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

[4.0] Edit links in dashboard modules title missing, Edit links in management views Subject missing #26322

Open
unknown-sudo opened this issue Sep 15, 2019 · 4 comments

Comments

@unknown-sudo
Copy link

On the home page of the backend there are several modules that display information.

  • Logged in User
  • Popular Articles
  • User actions (with edit user and edit plugin, etc. links)
  • Recently Added Articles
  • ...

In these modules the link is missing a title attribute, which describes to the user where the link is leading to. In the manage overviews there is a title attribute on the links, which is only called e.g. "edit test". I think it would be better if the text would display: "Edit Article with the title: test". / "Edit Plugin with the title: Content - Joomla"

@brianteeman
Copy link
Contributor

Sorry that's not correct and a common misconception

https://silktide.com/blog/2013/i-thought-title-text-improved-accessibility-i-was-wrong

@ghost ghost added the Backend Template label Sep 15, 2019
@ghost ghost changed the title (4.0) Edit links in dashboard modules title missing, Edit links in management views Subject missing [4.0] Edit links in dashboard modules title missing, Edit links in management views Subject missing Sep 15, 2019
@unknown-sudo
Copy link
Author

We're talking about two different things. I also say here that the title is completely missing. If it is intended and you can prove it for a better a11y, we must also remove it from the manage overviews.

@brianteeman
Copy link
Contributor

I think it would be better if the text would display: "Edit Article with the title: test". / "Edit Plugin with the title: Content - Joomla"

Dont you think that this is overly verbose?

I believe the thinking behind the current text was that it "made sense" in the context of the row. I appreciate that it makes less sense in the context of a list of links that a screen reader might generate

@brianteeman
Copy link
Contributor

I've done a PR for 3 of these (#27843). actionslogs is a bigger issue and needs a seperate pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants