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

Inspector tabs: text no longer visible when 'Show button text labels' preference is enabled #61947

Closed
afercia opened this issue May 24, 2024 · 3 comments · Fixed by #61949
Closed
Assignees
Labels
[Feature] Show button text labels A preference in the Post and Site Editor that makes buttons show text instead of icons [Package] Block editor /packages/block-editor [Type] Regression Related to a regression in the latest release

Comments

@afercia
Copy link
Contributor

afercia commented May 24, 2024

Description

This appears to be a regression from WordPress 6.5 and should be fixed soon.

Depending on the selected block, the Inspector panel may show content organized within tabs. By default these tabs use icon buttons. Example screenshot:

Screenshot 2024-05-24 at 15 48 21

When the 'Show button text labels' preference is enabled, the icons are supposed to be replacred by visible text. Example screenshot from WordPress 6.5. for a Navigation block:

6 5

On latest trunk, the text is not visible any longer. Screenshot for a Navigation block:

  trunk navigaton

Screenshot for an Image block:

trunk image

It appears that some recent changes to a CSS transition introduced this bug, as the ::before pseudo element that is supposed to generate the visible text has now an opacity: 0.

Worth noting the technique to reveal the text for the 'Show button text labels' preference is fragile to start with, as it uses pseudo elements that may be already used for other purposes. In this case, I think it's used also for the tabs focus style.

Regardless the change to the animation should be either fixed or reverted, as WP 6.6 Beta is very close.

Step-by-step reproduction instructions

  • Go to Options > Preferences > Accessibility, and enable the 'Show button text labels' preference.
  • Select any block that is supposed to render tabs in its Inspector panel e.g. an Image block or a Navigation block.
  • Observe the tabs text is no longer visible.

Screenshots, screen recording, code snippet

No response

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@afercia afercia added [Type] Regression Related to a regression in the latest release [Package] Editor /packages/editor [Feature] Show button text labels A preference in the Post and Site Editor that makes buttons show text instead of icons labels May 24, 2024
@afercia
Copy link
Contributor Author

afercia commented May 24, 2024

@jasmussen any clue what changed here? Thanks 🙏🏽

@jasmussen
Copy link
Contributor

No clue. @WordPress/gutenberg-core

@t-hamano
Copy link
Contributor

I've identified the problem and will submit a PR soon.

@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label May 24, 2024
@colorful-tones colorful-tones moved this to 🏗️ In Progress in WordPress 6.6 Editor Tasks May 24, 2024
@github-project-automation github-project-automation bot moved this from 🏗️ In Progress to ✅ Done in WordPress 6.6 Editor Tasks May 25, 2024
@afercia afercia added [Package] Block editor /packages/block-editor and removed [Package] Editor /packages/editor [Status] In Progress Tracking issues with work in progress labels May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Show button text labels A preference in the Post and Site Editor that makes buttons show text instead of icons [Package] Block editor /packages/block-editor [Type] Regression Related to a regression in the latest release
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants