Skip to content

refactor(theme): migrate content and menu bar tokens - #2444

Draft
spike-rabbit wants to merge 3 commits into
mainfrom
refactor/content-action-bar-system-tokens
Draft

refactor(theme): migrate content and menu bar tokens#2444
spike-rabbit wants to merge 3 commits into
mainfrom
refactor/content-action-bar-system-tokens

Conversation

@spike-rabbit

@spike-rabbit spike-rabbit commented Jul 29, 2026

Copy link
Copy Markdown
Member

Migrate content action bar and menu bar styling away from direct legacy semantic-token references.

Use the system-token mappings for menu bar surfaces and text states. Preserve the content action bar toggle width, as the dropdown padding token does not match its existing geometry.


Documentation.
Examples.
Dashboards Demo.
Playwright report.

Coverage Reports:

Code Coverage

@spike-rabbit spike-rabbit changed the title refactor(content-action-bar): migrate button token usage refactor(theme): migrate content and menu bar tokens Jul 29, 2026
&.hover {
color: variables.$element-action-secondary-text-hover;
background: variables.$element-action-secondary-hover;
color: system-tokens.$si-sys-text-accent-hover;

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.

on-secondary-hover

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated in d332106.

&[aria-expanded='true'] {
color: variables.$element-action-primary-active;
background: variables.$element-action-secondary-active;
color: system-tokens.$si-sys-text-accent-active;

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.

on-secondary-active

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Updated in d332106.

@spike-rabbit
spike-rabbit force-pushed the refactor/content-action-bar-system-tokens branch 2 times, most recently from d332106 to 6c94dc8 Compare July 29, 2026 13:38
@spike-rabbit
spike-rabbit force-pushed the refactor/content-action-bar-system-tokens branch from 6c94dc8 to 7873b3a Compare July 29, 2026 13:47
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