-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Sidebar Tabs: Add text only mode for block inspector tabs #46968
Conversation
Size Change: +962 B (0%) Total Size: 1.32 MB
ℹ️ View Unchanged
|
Flaky tests detected in 02bde25. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/3880223905
|
packages/block-editor/src/components/inspector-controls-tabs/use-inspector-controls-tabs.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks Aaron!
Fixes: #46871
What?
Display textual labels for the Block Inspector tabs when the "Show button text labels" preference is set.
Why?
It brings the inspector tabs more in line with the main editor and block toolbars when the "Show button text labels" option is set.
How?
Uses a similar CSS approach to the main and block toolbars. The difference is that the
aria-label
text is set as the content on the::before
pseudo element as::after
is already used for styling a TabPanel's active tab.Testing Instructions
Testing Instructions for Keyboard
Screenshots or screencast