You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The <BlockToolbar /> component should be able to be used directly with all of the functionality of the Block Toolbar. In its currents state, the <BlockToolbar /> component does not have many of the accessibility improvements and keyboard navigation features. Those are implemented via <NavigableToolbar /> within <BlockContextualToolbar />.
What is your proposed solution?
Refactor <BlockToolbar /> to include the features of <BlockContextualToolbar />. This would remove <BlockContextualToobar />
Remove <BlockContextualToobar /> and its private-api export
Replace usage of <BlockContextualToobar /> in edit-site, edit-post, edit-widget, etc headers with `
Refactor <SelectedBlockTools /> to use <BlockToolbar /> directly.
Rename <SelectedBlockTools /> to something that indicates its true purpose -- making the <BlockToolbar /> into a floating popover. Something like, <FloatingBlockToolbar /> or <BlockToolbarPopover />.
The text was updated successfully, but these errors were encountered:
Follow-up to #55787
What problem does this address?
The
<BlockToolbar />
component should be able to be used directly with all of the functionality of the Block Toolbar. In its currents state, the<BlockToolbar />
component does not have many of the accessibility improvements and keyboard navigation features. Those are implemented via<NavigableToolbar />
within<BlockContextualToolbar />
.What is your proposed solution?
<BlockToolbar />
to include the features of<BlockContextualToolbar />
. This would remove<BlockContextualToobar />
<BlockContextualToobar />
and its private-api export<BlockContextualToobar />
in edit-site, edit-post, edit-widget, etc headers with `<SelectedBlockTools />
to use<BlockToolbar />
directly.<SelectedBlockTools />
to something that indicates its true purpose -- making the<BlockToolbar />
into a floating popover. Something like,<FloatingBlockToolbar />
or<BlockToolbarPopover />
.The text was updated successfully, but these errors were encountered: