Skip to content
This repository has been archived by the owner on Dec 4, 2017. It is now read-only.

style-guide(add-a11y): Add accessibility quick wins #1854

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AlmeroSteyn
Copy link
Contributor

Adding accessibility recommendations to style guide.

These are all quick accessibility wins making use of HTML and CSS alone and requiring no use of ARIA.

@AlmeroSteyn AlmeroSteyn force-pushed the styleguide-a11y branch 2 times, most recently from 298fffa to 51a166a Compare July 10, 2016 13:04
+makeExample('style-guide/ts/10-01/app/index.avoid.html', 'page-lang', 'index.html')(avoid=1)

:marked
Provide the ISO language code as well the country code (where appropriate).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are not putting messages between avoid and do (correct me if I am wrong). We could put that as a Consider

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like we do. Let's hold on this and see if we can do better.

@AlmeroSteyn AlmeroSteyn force-pushed the styleguide-a11y branch 8 times, most recently from 36b8027 to 8043eee Compare July 11, 2016 18:40

.s-rule.avoid
:marked
**Avoid** totaly removing the focus outline on your page.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is totally a typo.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now it's completely fixed.

@Foxandxss
Copy link
Member

I like it, but it needs a bit of @wardbell magic on the English 👍

/cc @johnpapa.

@AlmeroSteyn
Copy link
Contributor Author

@Foxandxss: There is one flow issue now, because of number 10 being reserved for the router styles. Accessibility now shows as 10 on the menu above and then you end up in section 11. Will fix itself when the router styles are added.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants