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

Reusable blocks: the "Manage all..." link in the Inserter should look like a link #10009

Closed
afercia opened this issue Sep 18, 2018 · 7 comments
Closed
Assignees
Labels
[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes).

Comments

@afercia
Copy link
Contributor

afercia commented Sep 18, 2018

When there are saved reusable blocks, a link that looks like a button is displayed in the inserter, in the Reusable block section:

screen shot 2018-09-18 at 15 46 52

This is a link styled like a button. It just happened to me, using the keyboard, to press spacebar because it looks like a button and I've stared at the screen waiting for something to happen...
Of course, nothing happened because it's a link and it gets activated just with the Enter key.

This is a very good example of a design choice made just for visual purposes that is actually confusing for users. This is a link, it triggers navigation to an actual, different admin page (wp-admin/edit.php?post_type=wp_block) thus should look like a link.

Other improvements are certainly possible. For example, the discoverability of this control could be improved. Also, seems to me there's a bit of excessive title-casing: Manage All Reusable Blocks seems a bit too much to me.

For reference, see #8888 / #8927, and #4325 / #4471. See also the copy guidelines https://github.com/WordPress/gutenberg/blob/master/docs/reference/copy-guide.md#five-pay-attention-to-capitalization

Seems there's still a bit of inconsistency across the UI:

screen shot 2018-09-18 at 15 47 33

screen shot 2018-09-18 at 15 47 39

screen shot 2018-09-18 at 15 47 41

screen shot 2018-09-18 at 15 47 42

@afercia afercia added [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Needs Design Feedback Needs general design feedback. [Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) labels Sep 18, 2018
@afercia
Copy link
Contributor Author

afercia commented Sep 18, 2018

Worth noting in the "More" menu, it looks like a link (it also becomes blue on hover...)

manage all rb

Re: title casing, there are various inconsistencies here too, as only feature names should be title cased. Cases like Copy All Content are, honestly, a bit funny 🙂

@youknowriad
Copy link
Contributor

Also see #9989

@karmatosed
Copy link
Member

I want to just check I understand what is being suggested. The styling is not going to be changed - it will still be an icon - but it's how it's formatted?

@afercia
Copy link
Contributor Author

afercia commented Oct 9, 2018

The styling is not going to be changed

Icon-only controls are an anti-pattern for accessibility and should be avoided when possible. The a11y feedback is to change this in a textual link that looks like a link. That said, if the team doesn't want to change it then it's a responsibility of the team if this control is not fully accessible. Thanks.

@tofumatt
Copy link
Member

tofumatt commented Oct 9, 2018

Chiming in here from: from both an accessibility and usability perspective this should not look like a button. Currently that button is a really janky UX and it looking like a button makes me think clicking it will keep me in the editor context, but once I do click it (or press Enter), I'm prompted with Gutenberg's on-page-leave dialog because it's changing the page:

2018-10-09 18 02 57

That is super-unexpected from just a UX perspective. Additionally, there are accessibility concerns because I too expect that button to… act like a button 😄

That "button" is a link to another page on the site entirely and should be communicated as such. I don't see why we can't do:

screenshot 2018-10-09 18 09 19

That much more clearly communicates the action of clicking on that item.

@karmatosed
Copy link
Member

@tofumatt can we align to the right the link? It would fit in better I think with actions if that happened. Thanks for stepping in.

@tofumatt
Copy link
Member

tofumatt commented Oct 9, 2018

Works for me! I'll make the change in about an hour and update the screenshots.

gziolo pushed a commit that referenced this issue Oct 10, 2018
* fix: make "Manage Reusable Blocks" a link

Fix #10009

* Move text alignment to the right
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Synced Patterns Related to synced patterns (formerly reusable blocks) [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes).
Projects
None yet
Development

No branches or pull requests

4 participants