Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add aria-label to buttons in Sidebar #420

Closed

Conversation

mroeling
Copy link
Contributor

@mroeling mroeling commented Dec 7, 2017

First iteration: set the aria-label where also data-tip is used, using
the same text.

  • Set the aria-label on the three default buttons in the bottom of the
    sidebar
  • Set the aria-label on the TabItem component for the services.

Todo:

  • Think about better texts to aid the user instead of just re-using the
    labels

Motivation and Context
Based on the slack notification in #general @lanie Molinar, first try improve WIA-ARIA compliance

How Has This Been Tested?
Not yet... Was not succesfull to run Franz in dev mode on Windows.

Mark Roeling and others added 3 commits December 2, 2017 22:22
First iteration: set the aria-label where also data-tip is used, using
the same text.
* Set the aria-label on the three default buttons in the bottom of the
sidebar
* Set the aria-label on the TabItem component for the services.

Todo:
* Think about better texts to aid the user instead of just re-using the
labels
@stale
Copy link

stale bot commented Nov 16, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Nov 16, 2018
@adlk adlk removed wontfix labels Nov 19, 2018
@adlk
Copy link
Contributor

adlk commented Dec 10, 2018

Closing this in favor of #1166.

Thanks @mroeling for your initial push.

@adlk adlk closed this Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants