Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Split and update caret.js for Chromium changes #8147

Open
The-Compiler opened this issue Mar 27, 2024 · 0 comments
Open

Split and update caret.js for Chromium changes #8147

The-Compiler opened this issue Mar 27, 2024 · 0 comments
Labels
component: style / refactoring Issues related to coding styles or code that should be refactored. language: javascript Issues which require knowing JavaScript. priority: 2 - low Issues which are currently not very important.

Comments

@The-Compiler
Copy link
Member

Chromium has had some changes in recent years for its caret browsing code:

https://chromium.googlesource.com/chromium/src/+log/main/ui/accessibility/extensions/caretbrowsing (note, page takes an eternity to load)

Since the initial implementation in b184d2f:

3e1a26c Update copyright headers in ui/, url/ by Avi Drissman · 1 year, 6 months ago
34bb210 Refactor caretbrowsing/storage.js and add tests by Anastasia Helfinstein · 2 years, 1 month ago
28fa2dd Add accessibility extension test base by Anastasia Helfinstein · 2 years, 1 month ago
17e1505 Extract independent node/selection functions in caret browsing extension by Anastasia Helfinstein · 2 years, 2 months ago
abd3b15 Update comments in the caretenhancer extension by Anastasia Helfinstein · 2 years, 2 months ago
399e5ce Consolidate storage logic in caret browsing extension into a class by Anastasia Helfinstein · 2 years, 2 months ago
82b0912 Clean up caret browser extension for loops by Anastasia Helfinstein · 2 years, 2 months ago
a9952eef Migrate caret browser extension to manifest v3 by Anastasia Helfinstein · 2 years, 2 months ago
59c2a4a Convert caret browsing extension to an ES6 style class by Anastasia Helfinstein · 2 years, 2 months ago
05cb1be Update caret browsing extension to replace var with const/let by Anastasia Helfinstein · 2 years, 2 months ago
9857a74 Migrate Caret Browsing extension to use service workers by Anastasia Helfinstein · 2 years, 2 months ago
62b139f Add BUILD file for Caret Browsing extension, update one string. by Dominic Mazzoni · 4 years, 5 months ago

I also think we should aim to split the file into multiple JS files, to mirror the Chromium structure. Finally, we should probably avoid/revert style-only changes we did in comparison to upstream, to allow for easier diffing...

Since we diverge a lot, not sure yet what the best strategy for this would be. Try to reimplement Chromium's changes manually on top of ours? Start with the current Chromium codebase and try to get our changes in again?

@The-Compiler The-Compiler added component: style / refactoring Issues related to coding styles or code that should be refactored. priority: 2 - low Issues which are currently not very important. language: javascript Issues which require knowing JavaScript. labels Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: style / refactoring Issues related to coding styles or code that should be refactored. language: javascript Issues which require knowing JavaScript. priority: 2 - low Issues which are currently not very important.
Projects
None yet
Development

No branches or pull requests

1 participant