Skip to content

Fix Chrome 146+ detection (fixes https://github.com/yomidevs/yomitan/…#2358

Merged
Kuuuube merged 1 commit intoyomidevs:masterfrom
AuroraWright:fix_chrome_146_detection
Mar 29, 2026
Merged

Fix Chrome 146+ detection (fixes https://github.com/yomidevs/yomitan/…#2358
Kuuuube merged 1 commit intoyomidevs:masterfrom
AuroraWright:fix_chrome_146_detection

Conversation

@AuroraWright
Copy link
Copy Markdown

@AuroraWright AuroraWright commented Mar 29, 2026

Starting with Chrome 146, Chrome supports the "browser" namespace on top of the "chrome" one. This broke the yomitan browser detection causing Chrome to be detected as Firefox, in turn also breaking the clipboard monitor and possibly more side effects we haven't noticed.

@AuroraWright AuroraWright requested a review from a team as a code owner March 29, 2026 02:19
@Kuuuube Kuuuube added kind/bug The issue or PR is regarding a bug browser/chrome The issue is Chrome-only labels Mar 29, 2026
Copy link
Copy Markdown
Member

@Kuuuube Kuuuube left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Checked against https://developer.mozilla.org/en-US/docs/Web/HTTP/Reference/Headers/User-Agent/Firefox. Looks like all Firefox versions (besides ios which isnt even gecko iirc) have Firefox in the useragent string.

@Kuuuube Kuuuube added this pull request to the merge queue Mar 29, 2026
Merged via the queue into yomidevs:master with commit 08ba995 Mar 29, 2026
24 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

browser/chrome The issue is Chrome-only kind/bug The issue or PR is regarding a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants