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 - Buttons #40103

Merged
merged 7 commits into from
Mar 14, 2024
Merged

Conversation

deniskaber
Copy link
Contributor

Relates to #38811 , Milestone 2

Description

This migrates global CSS rules related to buttons to be locally scoped.

How to verify

  • Mainly visual and code comparison

@deniskaber deniskaber self-assigned this Mar 13, 2024
@deniskaber deniskaber added .Team/Embedding no-backport Do not backport this PR to any branch labels Mar 13, 2024
@deniskaber deniskaber requested a review from a team March 13, 2024 19:55
@metabase-bot metabase-bot bot added the visual Run Percy visual testing label Mar 13, 2024
Copy link

github-actions bot commented Mar 13, 2024

Codenotify: Notifying subscribers in CODENOTIFY files for diff f73b40c...b2d6570.

Notify File(s)
@alxnddr frontend/src/metabase/visualizations/components/PinMap.jsx
@ranquild frontend/src/metabase/setup/components/CompletedStep/CompletedStep.tsx
frontend/src/metabase/sharing/components/AddEditSidebar/DeleteSubscriptionAction.jsx
frontend/src/metabase/timelines/collections/components/TimelineDetailsModal/TimelineDetailsModal.tsx
frontend/src/metabase/timelines/collections/components/TimelineEmptyState/TimelineEmptyState.tsx

Copy link

replay-io bot commented Mar 13, 2024

Status In Progress ↗︎ 51 / 52
Commit b2d6570
Results
2362 Passed

@deniskaber deniskaber changed the title Migrate buttons Global to local css migration - Buttons Mar 13, 2024
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.

I think it should be good except this one button, which is a bit small but I'm not sure if this PR changed that (but it is a Button)

image

@oisincoveney oisincoveney self-requested a review March 14, 2024 12:19
Base automatically changed from 38811-no-global-css-migration-1 to master March 14, 2024 15:44
# Conflicts:
#	frontend/src/metabase/admin/databases/components/DatabaseList/DatabaseList.jsx
#	frontend/src/metabase/admin/datamodel/metadata/components/MetadataTableSettings/MetadataTableSettings.tsx
#	frontend/src/metabase/admin/settings/components/widgets/CustomGeoJSONWidget.jsx
#	frontend/src/metabase/components/DeleteModalWithConfirm/DeleteModalWithConfirm.jsx
#	frontend/src/metabase/core/components/Button/Button.tsx
#	frontend/src/metabase/css/core/colors.module.css
#	frontend/src/metabase/dashboard/components/AddSeriesModal/AddSeriesModal.tsx
#	frontend/src/metabase/dashboard/components/DashboardHeader/DashboardHeader.tsx
#	frontend/src/metabase/dashboard/components/DashboardHeader/DashboardHeaderView.tsx
#	frontend/src/metabase/query_builder/components/AlertModals.jsx
#	frontend/src/metabase/query_builder/components/DatasetEditor/DatasetEditor.jsx
#	frontend/src/metabase/query_builder/components/QueryVisualization.jsx
#	frontend/src/metabase/reference/components/EditHeader.jsx
#	frontend/src/metabase/reference/components/ReferenceHeader.jsx
#	frontend/src/metabase/sharing/components/AddEditSidebar/DeleteSubscriptionAction.jsx
#	frontend/src/metabase/visualizations/components/LegendItem.jsx
#	frontend/src/metabase/visualizations/components/PinMap.jsx
@deniskaber deniskaber added this to the 0.50 milestone Mar 14, 2024
@deniskaber deniskaber merged commit ac0bb15 into master Mar 14, 2024
107 of 108 checks passed
@deniskaber deniskaber deleted the 38811-no-global-css-buttons branch March 14, 2024 19:41
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 visual Run Percy visual testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants