Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

fix(svelte): Properly scope global CSS selector#63332

Merged
fkling merged 1 commit into
mainfrom
fkling/sk/fix-global-css-rule-tags-page
Jun 19, 2024
Merged

fix(svelte): Properly scope global CSS selector#63332
fkling merged 1 commit into
mainfrom
fkling/sk/fix-global-css-rule-tags-page

Conversation

@fkling

@fkling fkling commented Jun 19, 2024

Copy link
Copy Markdown
Contributor

The tags page contains a :global selector at the root level, which affects all elements that match the selector, even those not part of the page.
:global selectors always need to be scoped one way or the other.

Test plan

Code inspection, trivial change.

The tags page contains a `:global` selector at the root level, which
affects all elements that match the selector, even those not part of the
page.
`:global` selectors always need to be scoped one way or the other.
@fkling fkling requested review from a team June 19, 2024 08:01
@fkling fkling self-assigned this Jun 19, 2024
@cla-bot cla-bot Bot added the cla-signed label Jun 19, 2024
@fkling fkling merged commit 29e07b7 into main Jun 19, 2024
@fkling fkling deleted the fkling/sk/fix-global-css-rule-tags-page branch June 19, 2024 15:38

@camdencheek camdencheek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

good find!

@fkling

fkling commented Jun 19, 2024

Copy link
Copy Markdown
Contributor Author

It's unfortunate that it happened but luckily it's easy to search for :global.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants