Skip to content

[toolbar] Add disabled attribute and legend support - #1421

Merged
lukewarlow merged 1 commit into
openui:mainfrom
lukewarlow:toolbar-disabled-and-legend
Mar 19, 2026
Merged

[toolbar] Add disabled attribute and legend support#1421
lukewarlow merged 1 commit into
openui:mainfrom
lukewarlow:toolbar-disabled-and-legend

Conversation

@lukewarlow

@lukewarlow lukewarlow commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator

Adds a disabled attribute and child legend support to toolbar

  • first child legend will behave like in a fieldset, for naming the toolbar.
  • disabled attribute to disable all children (Except those within the legend)
  • UA styles to put toolbar legend on its own line above items.

Fixes #1399 and #1420

@lukewarlow
lukewarlow requested a review from scottaohara March 13, 2026 18:28
@lukewarlow
lukewarlow marked this pull request as draft March 13, 2026 18:28
@lukewarlow

lukewarlow commented Mar 13, 2026

Copy link
Copy Markdown
Collaborator Author

Need to update the styling that will maybe be needed to make this work nicely.

For now I've added styles that puts the first child legend as flex basis 100% so it's on its own line above the rest of the items.

@scottaohara scottaohara left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

looks good. some suggestions for you to take or leave.

Comment thread site/src/pages/components/toolbar.explainer.mdx Outdated
Comment thread site/src/pages/components/toolbar.explainer.mdx Outdated
@lukewarlow
lukewarlow force-pushed the toolbar-disabled-and-legend branch from 352f029 to 4d214fd Compare March 13, 2026 22:58
Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
@lukewarlow
lukewarlow force-pushed the toolbar-disabled-and-legend branch from 4d214fd to df0d7d2 Compare March 13, 2026 23:04
@lukewarlow
lukewarlow marked this pull request as ready for review March 13, 2026 23:05
@lukewarlow
lukewarlow merged commit d3462b2 into openui:main Mar 19, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[toolbar] Do we need a HTML way to label them?

2 participants