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

AG-11705 Update css variables hierarchy with theme params #3451

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

lsjroberts
Copy link
Member

@lsjroberts lsjroberts commented Jan 31, 2025

https://ag-grid.atlassian.net/browse/AG-11705

  • Adds accentColor, spacing and chrome theme params.
  • Updates css theme variables to inherit from the theme params and removes unnecessary indirection/inheritance.
  • Pushes some variables into their local context where they did not need to be global.
  • Removes references to grid variables that are no longer used.
  • Fixes inconsistent colours across context menu, dialogs and other popovers.

@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch from c37b2d8 to cf4c790 Compare February 3, 2025 09:27
@ag-grid ag-grid deleted a comment from github-actions bot Feb 3, 2025
@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch from cf4c790 to 6c8e05b Compare February 3, 2025 09:41
@ag-grid ag-grid deleted a comment from github-actions bot Feb 3, 2025
@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch 3 times, most recently from 6f699ae to 2cac269 Compare February 3, 2025 11:24
@ag-grid ag-grid deleted a comment from github-actions bot Feb 3, 2025
@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch from 2cac269 to 8da600f Compare February 3, 2025 11:35
@ag-grid ag-grid deleted a comment from github-actions bot Feb 3, 2025
@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch from 8da600f to 7a18092 Compare February 3, 2025 13:23
@ag-grid ag-grid deleted a comment from github-actions bot Feb 3, 2025
@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch 2 times, most recently from 175d471 to 3ebf0aa Compare February 3, 2025 14:51
@lsjroberts lsjroberts marked this pull request as ready for review February 3, 2025 15:45
@lsjroberts lsjroberts requested review from alantreadway and a team as code owners February 3, 2025 15:45
- Adds accentColor, spacing and chrome theme params.
- Updates css theme variables to inherit from the theme params and removes unnecessary indirection/inheritance.
- Pushes some variables into their local context where they did not need to be global.
- Fixes inconsistent colours across context menu, dialogs and other popovers.
@lsjroberts lsjroberts force-pushed the AG-11705-theme-params-defaults branch from 3ebf0aa to 796f7ff Compare February 3, 2025 16:32
Copy link
Member

@alantreadway alantreadway left a comment

Choose a reason for hiding this comment

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

LGTM

@alantreadway alantreadway merged commit d721580 into latest Feb 3, 2025
26 checks passed
@alantreadway alantreadway deleted the AG-11705-theme-params-defaults branch February 3, 2025 16:50
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