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

[7.3-stable] Make page select portable #3038

Merged
merged 4 commits into from
Sep 16, 2024
Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Sep 16, 2024

Backport

This will backport the following commits from main to 7.3-stable:

Questions ?

Please refer to the Backport tool documentation

The values where not correct

(cherry picked from commit 5f995e0)
We want to get rid of Sass variables and use custom properties instead.

(cherry picked from commit e0dbaa0)
By using it's own css file and include that in the component
we can use the page select outside of Alchemy UI (ie. Solidus).

Making use of custom properties with default values so that
the style can be overwritten if necessary.

(cherry picked from commit d998c56)
Some languages are very verbose, so we need more space for the parent page label

(cherry picked from commit d85fdce)
@tvdeyen tvdeyen requested a review from a team as a code owner September 16, 2024 14:22
@tvdeyen tvdeyen mentioned this pull request Sep 16, 2024
3 tasks
Copy link

codecov bot commented Sep 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (7.3-stable@0be49e8). Learn more about missing BASE report.

Additional details and impacted files
@@              Coverage Diff              @@
##             7.3-stable    #3038   +/-   ##
=============================================
  Coverage              ?   96.47%           
=============================================
  Files                 ?      237           
  Lines                 ?     6405           
  Branches              ?        0           
=============================================
  Hits                  ?     6179           
  Misses                ?      226           
  Partials              ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tvdeyen tvdeyen enabled auto-merge September 16, 2024 14:35
@tvdeyen tvdeyen disabled auto-merge September 16, 2024 14:35
@tvdeyen tvdeyen merged commit a66156a into 7.3-stable Sep 16, 2024
36 checks passed
@tvdeyen tvdeyen deleted the backport/7.3-stable/pr-3037 branch September 16, 2024 14:36
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