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

Permalink and its Edit button could be clearer #15298

Closed
karlgroves opened this issue Apr 30, 2019 · 7 comments
Closed

Permalink and its Edit button could be clearer #15298

karlgroves opened this issue Apr 30, 2019 · 7 comments
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Needs Dev Ready for, and needs developer efforts

Comments

@karlgroves
Copy link

Permalink and its Edit button could be clearer

  • Severity: Low
  • Affected Populations:
    • Blind
    • Cognitively Impaired
  • Platform(s):
    • All / Universal
  • Components affected:
    • Title Block

Issue description

The "Permalink" section of the Title Block has an "Edit" button
which is not semantically associated with the permalink except by
context. Similarly, the permalink itself has the word "Permalink:"
before it, but this is not explicitly associated either.

Although association by context is generally sufficient for content like
this, it would nevertheless be helpful to add a more descriptive label
to the button, and to describe the fact that the link is a permalink
within its screen reader text.

These changes would help to make the meaning clearer to screen reader
users, especially those with a cognitive disability.

Issue Code
    <span class="editor-post-permalink__label">Permalink:</span>


    <a target="_blank" class="..." href="http://142.93.114.99/?p=19" rel="external noreferrer noopener">


        http://142.93.114.99/2019/01/31/gutenberg-editor-demo-edited-for-output-scoping/‎


        <span class="screen-reader-text">(opens in a new tab)</span>


        ...


    </a>


    <button type="button" class="...">Edit</button>

Remediation Guidance

Add an aria-label to the "Edit" button which explains what it does
more clearly, and amend the .screen-reader-text of the permalink to
do the same.

Recommended Code
    <span class="editor-post-permalink__label">Permalink:</span>


    <a target="_blank" class="..." href="http://142.93.114.99/?p=19" rel="external noreferrer noopener">


        http://142.93.114.99/2019/01/31/gutenberg-editor-demo-edited-for-output-scoping/‎


        <span class="screen-reader-text">(permalink opens in a new tab)</span>


        ...


    </a>


    <button aria-label="Edit permalink" type="button" class="...">Edit</button>

Relevant standards

Note: This issue may be a duplicate with other existing accessibility-related bugs in this project. This issue comes from the Gutenberg accessibility audit, performed by Tenon and funded by WP Campus. This issue is GUT-87 in Tenon's report

@gziolo gziolo added the Needs Accessibility Feedback Need input from accessibility label Apr 30, 2019
@afercia
Copy link
Contributor

afercia commented May 1, 2019

Totally agree. The current permalink UI has been discussed several times and it was mentioned as "candidate for removal" a few times, see for example #10996 (comment). Previously: #5494

Related: #12031

@afercia afercia added [Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). and removed Needs Accessibility Feedback Need input from accessibility labels May 1, 2019
@melchoyce
Copy link
Contributor

Screenshot from full report:

image

@karmatosed karmatosed added the Needs Design Feedback Needs general design feedback. label May 6, 2019
@mapk
Copy link
Contributor

mapk commented May 14, 2019

Design team talked about this in today's triage. Let's implement the suggested changes.

  1. Add an aria-label to the "Edit" button.
  2. Amend the .screen-reader-text of the permalink to explain clearly what it does.

I'm removing the needs design feedback label and adding needs dev.

@mapk mapk added Needs Dev Ready for, and needs developer efforts and removed Needs Design Feedback Needs general design feedback. labels May 14, 2019
@afercia
Copy link
Contributor

afercia commented May 14, 2019

As previously commented, these changes are very welcome. However, this UI has other important issues, starting from its placement in the DOM and the tab order as outlined in #5494.

Either these other issues should be solved or this UI should be finally removed in favor of the permalink UI in the sidebar.

@mapk
Copy link
Contributor

mapk commented May 15, 2019

Good point @afercia. I think we can get this work completed first, and continue discussion and work around #11553 which involves possibly moving the permalink to the Inspector.

@afercia
Copy link
Contributor

afercia commented May 15, 2019

Fine, then I'm going to reopen #5494.

@Soean
Copy link
Member

Soean commented Mar 26, 2020

The permalink was removed from the title in #21099. Now it's only in the sidebar. Can we close this issue?

@Soean Soean closed this as completed Mar 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Focus] Accessibility (a11y) Changes that impact accessibility and need corresponding review (e.g. markup changes). Needs Dev Ready for, and needs developer efforts
Projects
None yet
Development

No branches or pull requests

7 participants