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

Global to local css migration - grabber #40007

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

deniskaber
Copy link
Contributor

@deniskaber deniskaber commented Mar 12, 2024

Relates to #38811 , Milestone 2

Description

This migrates global CSS rules from grabber.module.css to be locally scoped.

How to verify

  • Mainly visual and code comparison

@deniskaber deniskaber self-assigned this Mar 12, 2024
@deniskaber deniskaber force-pushed the 38811-no-global-css-migration-1 branch from ffdad00 to 9cc0e08 Compare March 13, 2024 19:44
@deniskaber deniskaber force-pushed the 38811-no-global-css-migration-2 branch 2 times, most recently from 2cc6eaa to ea46466 Compare March 14, 2024 12:56
@deniskaber deniskaber changed the base branch from 38811-no-global-css-migration-1 to 38811-no-global-css-query-builder March 14, 2024 12:57
@deniskaber deniskaber changed the title 38811 no global css migration 2 38811 no global css migration - grabber Mar 14, 2024
@deniskaber deniskaber added the no-backport Do not backport this PR to any branch label Mar 14, 2024
@deniskaber deniskaber marked this pull request as ready for review March 14, 2024 12:58
Copy link

replay-io bot commented Mar 14, 2024

Status Complete ↗︎
Commit 277c784
Results
⚠️ 3 Flaky
2368 Passed

@deniskaber deniskaber requested a review from a team March 14, 2024 19:57
@deniskaber deniskaber added this to the 0.50 milestone Mar 14, 2024
@deniskaber deniskaber changed the title 38811 no global css migration - grabber Global to local css migration - grabber Mar 14, 2024
@@ -100,6 +100,7 @@

/* white - move to bottom for specificity since its often used on hovers, etc */
:global(.text-white),
.text-white,
Copy link
Contributor

Choose a reason for hiding this comment

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

do we still need to add this one?

Copy link
Contributor

@oisincoveney oisincoveney left a comment

Choose a reason for hiding this comment

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

just the one question (plus tests passing obv), but looks good otherwise

@deniskaber deniskaber force-pushed the 38811-no-global-css-migration-2 branch from 8e759b6 to 2d348ce Compare March 18, 2024 20:29
@deniskaber deniskaber changed the base branch from 38811-no-global-css-query-builder to master March 18, 2024 20:29
@deniskaber deniskaber merged commit 2fd5d6e into master Mar 18, 2024
107 checks passed
@deniskaber deniskaber deleted the 38811-no-global-css-migration-2 branch March 18, 2024 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-backport Do not backport this PR to any branch .Team/Embedding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants