madeyourday / contao-rocksolid-custom-elements

RockSolid Custom Elements Contao Extension
http://rocksolidthemes.com/de/contao/plugins/custom-content-elements
MIT License
48 stars 12 forks source link

Please add alt-Attributes to toolbar-icons #84

Closed NinaG closed 7 years ago

NinaG commented 7 years ago

At the moment alt-attributes are empty in toolbar-icons: <div class="rsce_list_toolbar"><a href="" class="rsce_list_toolbar_up" onclick="rsceMoveElement(this, -1);return false;" title="" data-rsce-title="Eine Position nach oben verschieben"><img src="system/themes/flexible/icons/up.svg" alt="" width="13" height="16"></a> <a href="" class="rsce_list_toolbar_down" onclick="rsceMoveElement(this, 1);return false;" title="" data-rsce-title="Eine Position nach unten verschieben"><img src="system/themes/flexible/icons/down.svg" alt="" width="13" height="16"></a> <img src="system/themes/flexible/icons/drag.svg" alt="" class="drag-handle rsce_list_toolbar_drag" title="" width="16" height="16"> <a href="" class="rsce_list_toolbar_delete" onclick="rsceDeleteElement(this);return false;" title="" data-rsce-title="Element löschen"><img src="system/themes/flexible/icons/delete.svg" alt="" width="16" height="16"></a> <a href="" class="rsce_list_toolbar_new" onclick="rsceNewElementAfter(this);return false;" title="" data-rsce-title="Neues Element erstellen"><img src="system/themes/flexible/icons/new.svg" alt="" width="16" height="16"></a> </div>

Please change it to this (filled ALT attributes): <div class="rsce_list_toolbar"><a href="" class="rsce_list_toolbar_up" onclick="rsceMoveElement(this, -1);return false;" title="" data-rsce-title="Eine Position nach oben verschieben"><img src="system/themes/flexible/icons/up.svg" alt="Eine Position nach oben verschieben" width="13" height="16"></a> <a href="" class="rsce_list_toolbar_down" onclick="rsceMoveElement(this, 1);return false;" title="" data-rsce-title="Eine Position nach unten verschieben"><img src="system/themes/flexible/icons/down.svg" alt="Eine Position nach unten verschieben" width="13" height="16"></a> <img src="system/themes/flexible/icons/drag.svg" alt="" class="drag-handle rsce_list_toolbar_drag" title="" width="16" height="16"> <a href="" class="rsce_list_toolbar_delete" onclick="rsceDeleteElement(this);return false;" title="" data-rsce-title="Element löschen"><img src="system/themes/flexible/icons/delete.svg" alt="Element löschen" width="16" height="16"></a> <a href="" class="rsce_list_toolbar_new" onclick="rsceNewElementAfter(this);return false;" title="" data-rsce-title="Neues Element erstellen"><img src="system/themes/flexible/icons/new.svg" alt="Neues Element erstellen" width="16" height="16"></a> </div>

Without this, it is very hard for screen reader users to understand and use the icons.

Thank you for your help!

ausi commented 7 years ago

Fixed. Thank you for reporting this issue.