? Success

User tests: Successful: Unsuccessful:

avatar smehrbrodt
smehrbrodt
28 Mar 2015

The user list looks quite cluttered with a button "Add a note" being below each user:

User list before

This change removes the text from that button (the text is still shown on a tooltip).

How to test:

Go to user manager. Without any notes attached to a user, it should look like this:

after

Then, add a note to a user. Before it looked like this:

before

After applying the patch, it should look like this (the tooltips should still work):

after

avatar smehrbrodt smehrbrodt - open - 28 Mar 2015
avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2015
Labels Added: ?
avatar zero-24 zero-24 - change - 28 Mar 2015
Easy No Yes
avatar zero-24 zero-24 - change - 28 Mar 2015
Category Templates (admin) UI/UX
avatar wilsonge
wilsonge - comment - 29 Mar 2015

I think this is a bad idea. Tooltips are pretty inaccessible and I know the accessibility group is working to remove many of the tooltips currently used in the CMS. So I think this PR is taking things in the wrong direction

avatar n9iels
n9iels - comment - 29 Mar 2015

It looks good, but I agree with @wilsonge. If the tooltip is not shown, it is not clear what kind of functions the icons have.

avatar losedk
losedk - comment - 1 Apr 2015

When I did #6264 I had the same idea of removing the text from the buttons. But decided that it would be bad for accessibility

avatar wilsonge
wilsonge - comment - 1 Apr 2015

On the basis of the above comments I'm going to close this PR

avatar wilsonge wilsonge - change - 1 Apr 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-04-01 11:48:43
avatar wilsonge wilsonge - close - 1 Apr 2015
avatar wilsonge wilsonge - close - 1 Apr 2015
avatar smehrbrodt
smehrbrodt - comment - 1 Apr 2015

Why? I don't understand the problem with accessiblity. If there is a problem, we should use the aria-label attribute, not just show a label everywhere

avatar rdeutz
rdeutz - comment - 1 Apr 2015

@smehrbrodt if you have a problem with coordination and you can't control the mouse pointer exactly over the button then you are not able to get the info.

Add a Comment

Login with GitHub to post a comment