Skip to content

fix: recognise Vivaldi on install and leave its side panel alone - #1215

Merged
ankit merged 3 commits into
mainfrom
claude/chromium-browser-compat-a59f6c
Oct 15, 2026
Merged

ankit merged 3 commits into
mainfrom
claude/chromium-browser-compat-a59f6c

Conversation

@ankit

@ankit ankit commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner

Stylebot now recognises Vivaldi on install and update, not only after its side panel first fails.

  • Detect Vivaldi by the vivExtData field on windows, so the Side panel choice is hidden from the start.
  • Skip sidePanel.setOptions in Vivaldi, so the panel it opens on install and update is blank instead of its start page.

Review notes: Vivaldi still opens that panel (its bug). Making sidePanel optional stops it but crashes Chrome on tabs open before the grant, so it stays required.

Testing: unit, typecheck, lint, side panel and editor window e2e; checked by hand in Vivaldi.

🤖 Generated with Claude Code

ankit and others added 2 commits October 14, 2026 20:31
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Bundle size

This PR changes what the popup or content scripts load.

Bundle Size Change Gzip Change
Popup 178.9 KB +608 B (0.3%) 48.8 KB +106 B (0.2%)
fonts/fonts.css 2.9 KB – 458 B –
popup/index.css 23.3 KB – 5.2 KB –
popup/index.js 152.7 KB +608 B (0.4%) 43.2 KB +106 B (0.2%)
Content script (document_start) 90.0 KB +608 B (0.7%) 23.2 KB +91 B (0.4%)
editor/index.js 50.8 KB +608 B (1.2%) 11.9 KB +91 B (0.8%)
inject-css/index.js 36.1 KB – 9.8 KB –
site-bridge/index.js 3.1 KB – 1.4 KB –

@ankit
ankit merged commit cac120d into main Oct 15, 2026
7 checks passed
@ankit
ankit deleted the claude/chromium-browser-compat-a59f6c branch October 15, 2026 13:09
@ankit ankit mentioned this pull request Oct 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant