ARIA: issue with Inline `#clr-color-marker` about coloris HOT 6 OPEN

Indeed, role="presentation" is the same as role="none", so it shouldn't be applied to interactive elements.

I think for now we can safely ignore the warning from FastPass. I'm more concerned with how screen readers treat the UI elements than with strictly complying with the standards. I'm open to any suggestions to improve compliance though.

from coloris.

I did consider what ARIA role to add and I couldn't find anything that fits. There is an open issue related to this on the ARIA Github repo that I've been watching for a while now (w3c/aria#432).

I haven't actually tested the impact of adding role="presentation" to #clr-color-marker on the popular screen readers (JAWS, VoiceOver, NVDA, ...). If you are willing to test it to make sure it doesn't have any negative impact on usability, I'll happily add the role.

from coloris.

@melloware Maybe we can ask @christophs78 to test the change.

from coloris.

@mdbassit you are right role="presentation" is not correct as the docs say

Note: The element with role="presentation" is not part of the accessibility tree and should not have an accessible name. Do not use aria-labelledby or aria-label.

from coloris.

Agreed should we leave this open for tracking and so no one else re-reports it?

from coloris.

Alright, let's leave it open for now

from coloris.