Opened 10 years ago
Closed 4 weeks ago
#36447 closed enhancement (fixed)
Responsive preview icons in Customizer need visible text
| Reported by: | ahortin | Owned by: | joedolson |
|---|---|---|---|
| Priority: | normal | Milestone: | 7.1 |
| Component: | Customize | Version: | 4.6 |
| Severity: | normal | Keywords: | has-screenshots tooltips has-patch a11y-visible-labels commit |
| Cc: | Focuses: | accessibility |
Description
The new icons at the bottom of the Customizer for toggling the preview window of your site really need tooltips to indicate what they're for.
Just like the tooltips on the Visual Editor icons, other icons in the Dashboard should have tooltips as well. As leading usability expert Jakob Nielsen explains;
A user’s understanding of an icon is based on previous experience. Due to the absence of a standard usage for most icons, text labels are necessary to communicate the meaning and reduce ambiguity.
https://www.nngroup.com/articles/icon-usability/
Even the Google Design Guidelines recommend tooltips for icons
https://www.google.com/design/spec/components/tooltips.html#
I originally raised this as a post on the Beta forum but it was suggested that since it's getting late in the 4.5 release cycle it would be best to raise it as a Trac ticket.
Attachments (7)
Change History (40)
#2
@
10 years ago
Repeating my comment from Slack:
Because these buttons perform non-destructive actions that are visually obvious when clicked, I think the simplicity of not having tooltips is better. They would certainly become distracting once the user knows what the buttons do, and after using them a few times I would expect most users to remember their purpose. My only concern would be whether users try clicking them in the first place, and that could potentially be solved with a feature pointer or by the specific way it's highlighted on the about page.
I also wouldn't want to attach text to these as the specific buttons are intentionally ambiguous - the previewed sizes are arbitrary and not intended to represent specific devices, only provide a general idea of how the site behaves on different screen sizes.
If appropriately vague text labels could be found, we could look at doing this. I personally feel that the labels there for screen readers are a bit too specific (potentially mis-interpreted) already.
Full conversation: https://wordpress.slack.com/archives/core-customize/p1460024296000353.
#3
@
10 years ago
@celloexpressions the fact that they're non-destructive is irrelevant. People shouldn't have to click a button to find out what it does. That's the most annoying thing ever. Why is there a tooltip on the Distraction Free Writing icon in the Visual Editor? It's non-desctructive as well. By your way of thinking, this tooltip should be removed also.
You should know what an icon/button does BEFORE you click it. If I'm using an interface and I don't know what a particular button does, I'm less likely to click it at all, just in case it messes up my work. If I don't know what the icon/button does, how do I know if it's non-destructive?
Even though the sizes may be arbitrary, they'd still benefit from having labels like "Mobile view/Tablet view/Desktop view", or something along those lines. They're not device specific and will at least give people an idea of what the icon does.
Also, tooltips don't become distracting once you know what an icon/button does. I refer to the Visual Editor again. I don't find the tooltips on any of those buttons annoying and I've been using them for 10+ years. Once you know what an icon does, you simply click it and move on. Your mouse doesn't tend to hover over it. The tooltip will only display for a microsecond.
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
10 years ago
#5
@
10 years ago
- Keywords needs-screenshots added
- Milestone Awaiting Review → Future Release
- Owner set to
- Status new → assigned
Discussed this ticket in the wpa11y meeting:
Adding Tooltips seems like a good idea:
@iamjolly: I really like how GitHub does the icons with aria-label and tooltips.
This ticket was mentioned in Slack in #accessibility by iamjolly. View the logs.
10 years ago
#8
@
9 years ago
- Keywords ui-feedback added
Just to give this a bit of a push forward. I jacked some of the Github code and created a .tooltip class (limited to #customize-controls for now). Would definitely appreciate some design input on these things to help them match the admin look better.
One complication that I'm running into with these CSS-only tooltips is that when one of those buttons is clicked the tooltip sticks around. I think there are JS solutions to fix that (like https://marcysutton.com/button-focus-hell/), but I'm a bit too overwhelmed by the amount of Customizer JS to make that change properly.
Will add a patch with what I have so far, which adds the tooltip class CSS and adds the class to the device buttons.
This ticket was mentioned in Slack in #core-customize by westonruter. View the logs.
9 years ago
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
This ticket was mentioned in Slack in #design by joshuawold. View the logs.
8 years ago
#13
@
8 years ago
We just discussed this in our #design triage. As long as the tooltips are created the same way as "tooltips" for the WordPress Gutenberg editor (with accessibility in mind), then it definitely makes sense to include in the Customizer.
Also, the text suggested seems to work well.
This ticket was mentioned in Slack in #design by karmatosed. View the logs.
7 years ago
This ticket was mentioned in Slack in #design by chaion07. View the logs.
5 years ago
#16
@
5 years ago
- Keywords dev-feedback needs-patch added; ui-feedback removed
This ticket has had design recommendations from 3 years ago. Can a developer please pick it up from here and add a patch with screenshots to see the patch working
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
17 months ago
This ticket was mentioned in PR #8593 on WordPress/wordpress-develop by @abcd95.
17 months ago
#19
- Keywords has-patch added; needs-patch removed
Trac ticket:
#21
@
12 months ago
The proposed implementation (PR 8593) has an issue with z-index, but I wonder if this is the right approach because we already have some tooltips, and they are powered by JS and already have some styling (check the screenshots above).
@joedolson What the approach should be taken in general?
#22
@
11 months ago
- Milestone 6.9 → Future Release
Punting this again.
@oglekler What we'd ideally like to do is implement tooltips using native CSS popovers, but we're waiting on Firefox to complete their implementation.
There are tooltips already, but they're only in the editor, and not readily available. What you're pointing to is just a title attribute, and I don't think there's any JS or custom styling involved there.
#23
@
4 months ago
- Milestone Future Release → 7.1
With CSS anchor positioning having meet the basic criteria for WordPress browser support, let's go ahead and work on a new patch to get this resolved using native tooltips.
#24
@
7 weeks ago
- Keywords a11y-visible-labels added
From an accessibility (and usability) perspective, tooltips are anyways a compromise. They don't solve all the problems.
The most accessible apps and software do use both icons and visible labels. Simply because:
- Some users are blind to icons of they don't understand them. They prefer text.
- Some users struggle with text and prefer icons.
From an accessibility perspective, we should establish a principle and that's use bothicons and visible labels, whenever possible.
I'm attaching a screenshot to illustrate a quick experiment I made by editing only the related CSS for these icon buttons.
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
6 weeks ago
#26
@
4 weeks ago
As 7.1 Beta1 is approaching, I would like to check the status of this ticket. My understanding is that this ticket depends on #51006, and the mechanism using native tooltips needs to be implemented in the core first.
This ticket was mentioned in PR #12519 on WordPress/wordpress-develop by @afercia.
4 weeks ago
#27
#28
@
4 weeks ago
In https://github.com/WordPress/wordpress-develop/pull/12519 I'm proposing to make the device button labels visible, shortened to:
- Desktop
- Tablet
- Mobile
I'd think the shortened labels are sufficiently clear and there is no need to specify the enter a preview mode.
This approach also aims to establish a few principles:
- Tooltips are a compromise anyways. Visible labels should always be preferred wherever possible.
- The most accessible web and desktop applications do use a combination of icon + visible label to take into account the widest spectrum of user needs.
The existing layout and CSS already take into account longer translations. When the 'Hide controls' label or the buttons labels are longer than in English, the layout doesn't break. See attached screenshot.
@joedolson if you think this approach is acceptable, I would suggest to merge this enhancement soon. The only doubt I have is that the label text size must necessarily be pretty small.
To test:
- Set Twenty Twenty-One as the active theme.
- Access the Customizer and observe the device preview buttons at the bottom of the left panel.
- Same in a Theme preview from WP Admin > Appearance > Themes > Live Preview
#29
@
4 weeks ago
These controls are fairly large already, so having room for that switching makes sense to me. Visible labels are definitely preferred, and less distracting than adding tooltips.
I agree with you that the very small text isn't great, but the words are common and supported by associated images, so the text isn't really *critical*. And given that these controls haven't had any visible text up to now, it's a clear improvement over what's already there.
I also agree with simplifying the labels; all of that extra verbiage to describe what the buttons do should be clear from context, especially given how common responsive controls are at this point.
The only change I think I'd consider is switching div class="devices-wrapper" to have role="menu" and an aria-label for context. I can push that change up.
#30
@
4 weeks ago
I'm reconsider role="menu"; I think just marking as a group is sufficient. I don't think we want to go to the extensive changes required to change the keyboard navigation.
#31
@
4 weeks ago
- Keywords commit added; dev-feedback removed
With the addition of the group role, I think that the context is sufficient, and this is commit-ready.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)

+1
I personally feel the tooltips would be helpful. The icons aren't totally clear what the buttons do. Here the accessible text is even more clear than the visual icons, I think. Here the visual users need an accessibility aid.