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

Fix Anki css scoping breaking due to Anki shipping with incredibly out of date Chromium version (112) #1747

Merged
merged 2 commits into from
Jan 5, 2025

Conversation

Kuuuube
Copy link
Member

@Kuuuube Kuuuube commented Jan 4, 2025

Much more robust than the previous regex solution.

Related: #1500 #1726

@Kuuuube Kuuuube added kind/bug The issue or PR is regarding a bug area/anki The issue or PR is related to Anki integration regression This issue or PR is related to a regression labels Jan 4, 2025
@Kuuuube Kuuuube requested a review from a team as a code owner January 4, 2025 20:56
Copy link

github-actions bot commented Jan 4, 2025

@Kuuuube
Copy link
Member Author

Kuuuube commented Jan 4, 2025

Tests are breaking because node doesn't support the JS I've used (CSSStyleSheet and specifically, it fails on replaceSync). All major browsers do support it so there's no worry of using it in Yomitan from that angle.

No clue why a single playwright test on the popup failed (all others succeeded). Might be unrelated to this PR.

Unsure what to do about this.

@Kuuuube Kuuuube added this pull request to the merge queue Jan 5, 2025
Merged via the queue into yomidevs:master with commit 2b08084 Jan 5, 2025
25 checks passed
@Kuuuube Kuuuube deleted the legacy-css-scope branch January 5, 2025 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/anki The issue or PR is related to Anki integration kind/bug The issue or PR is regarding a bug regression This issue or PR is related to a regression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants