selfoss icon indicating copy to clipboard operation
selfoss copied to clipboard

Found many accessibility violations with quick accessibility check

Open akash07k opened this issue 6 years ago • 7 comments

First, let's know that what's the accessibility? https://www.w3.org/WAI/fundamentals/accessibility-intro/ https://webaim.org/intro/ So, accessibility is really the important part of everything in todays world and it is really needed. I've evaluated the basic accessibility of the selfoss driven website and there are many accessibility issues found with the quick accessibility pass itself. I've done this tast pass with the opensource tool by microsoft which is named: 'accessibility insights for web' https://accessibilityinsights.io/docs/en/web/overview https://chrome.google.com/webstore/detail/accessibility-insights-fo/pbjjkligggfmakdaogkfomddhfmpjeni Please find the attached test pass results with more details

AutomatedChecks_20190916_Readerotica1281.docx

akash07k avatar Sep 16 '19 11:09 akash07k

Thanks for reporting. Accessibility is something I am always trying to improve in selfoss, most of the UI should be navigable by keyboard and I also tried using it blindfolded (though I only have access to Orca screen reader). Overall I would say it is usable, though there are still many small annoyances in need of a fix.

I fixed some of the issues in #1133 but will leave the contrast issues to a design brush up later in the cycle.

jtojnar avatar Sep 16 '19 19:09 jtojnar

@jtojnar Thanks a lot for your efforts bro. Actually, there are couple of more things, such as, the landmarks should be correctly implemented, skip links should be properly implemented and all the shortcut keys should be exposed by using accesskey or aria attributes to the screen reader users. Also, I've read in #1028 That you need some help with aria live regions? so, please let me know that how can I help? I'll try to become familiarise with the selfoss code myself and will try to fix somethings alongside. BTW, selfoss is the best!

akash07k avatar Sep 17 '19 03:09 akash07k

Today I'll pull all your patches and will provide more feedback on this.

akash07k avatar Sep 17 '19 03:09 akash07k

And yes, we should try to expose the control type of every control by using role="valid role" if it is defined in a div. If you have the time, then you can go through these links:

https://support.siteimprove.com/hc/en-gb/articles/115003457351-How-skip-links-can-help-with-website-accessibility https://webaim.org/techniques/skipnav/ https://accessibility.oit.ncsu.edu/it-accessibility-at-nc-state/developers/accessibility-handbook/aria-landmarks/ https://accessibility.oit.ncsu.edu/using-aria-landmarks-a-demonstration/ https://dequeuniversity.com/assets/html/jquery-summit/html5/slides/landmarks.html

akash07k avatar Sep 17 '19 03:09 akash07k

We can add aria-live regions for the instances where we refresh the sources, add the sources, edit the sources etc. Also, the aria-expanded state should be associated with the every feed heading because every feed heading is collapsable and expandable. With this, screen reader user will be able to identify that the feed is expanded or collapsed. And, since, the feed heading is always clickable, the role="link" should be used on that along with aria-expanded"

akash07k avatar Sep 17 '19 03:09 akash07k

Are any of these still relevant?

jtojnar avatar Oct 04 '19 02:10 jtojnar

I will run the accessibility check today again and will let you know

On 10/4/19, Jan Tojnar [email protected] wrote:

Are any of these still relevant?

-- You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub: https://github.com/SSilence/selfoss/issues/1132#issuecomment-538199837

akash07k avatar Oct 04 '19 03:10 akash07k