Simply start acting: Yet another button, not a link

joomla / accessibility

Collaboration to implement accessbility into Joomla

8 stars 12 forks source link

Simply start acting: Yet another button, not a link #57

Closed zwiastunsw closed 4 years ago

zwiastunsw commented 5 years ago

Hi all. This is the next idea for simple PR. Who will do it?

Improve accessibility of button Print in Print modal window

In the modal window there is a link Print. This link looks like a button, but it's a CSS styling.

Of course, it should be a button, not a link. You need a simple patch in the code. Path to file: administrator/components/com_content/Service/HTML/Icon.php

tarot-ray commented 5 years ago

administrator/components/com_content/Service/HTML/Icon.php ??? This path doesn't exist. There is no 'Service' directory under the administrator/components/com_content/ directory.

brianteeman commented 5 years ago

@tarot-ray yes there is - are you looking at the wrong branch?

tarot-ray commented 5 years ago

@brianteeman Yes, I was. Sorry for that. I made a PR for it.

wilsonge commented 4 years ago

This got removed from J4 so nothing left to do here