Skip to content

feat: isolate styles from page contents#8

Merged
dustin-jw merged 6 commits intomainfrom
feat--isolate-styles
Jan 27, 2026
Merged

feat: isolate styles from page contents#8
dustin-jw merged 6 commits intomainfrom
feat--isolate-styles

Conversation

@dustin-jw
Copy link
Contributor

@dustin-jw dustin-jw commented Nov 11, 2025

Description

This is meant to prevent page styles from interfering with the panel's styles by reverting all properties within the panel, then setting whatever styles we need. Sites that change the root font size will cause trouble, such as this one, but there's not much we can do to mitigate that (it's bad practice anyway). There may be cases where the specificity of a page's styling for a particular element is higher than the selector that reverts the page styles, but those seem to be rare as far as I can tell.

To Validate

  1. Pull down this branch
  2. Run npm run build
  3. Load the extension in Chrome or Firefox
  4. For a random, somewhat comprehensive sample of sites, run the extension and confirm that there are no unintended styles leaking through from the page

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

📦 Extension packages built successfully!

Download your Chrome and Firefox extension packages

@dustin-jw dustin-jw force-pushed the feat--isolate-styles branch 2 times, most recently from 1c26e8e to 6199d59 Compare December 2, 2025 19:02
@dustin-jw dustin-jw force-pushed the feat--isolate-styles branch from 6199d59 to 8343167 Compare December 16, 2025 18:35
@dustin-jw dustin-jw marked this pull request as ready for review December 16, 2025 18:42
@dustin-jw dustin-jw force-pushed the feat--isolate-styles branch 2 times, most recently from 917993b to 3af05bc Compare January 13, 2026 18:50
@dustin-jw dustin-jw force-pushed the feat--isolate-styles branch from 3af05bc to d6b0635 Compare January 27, 2026 17:57
@dustin-jw dustin-jw merged commit c0a5faf into main Jan 27, 2026
1 check passed
@dustin-jw dustin-jw deleted the feat--isolate-styles branch January 27, 2026 18:37
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.

2 participants