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

Improve Visualize sidebar accessibility. #11754

Merged

Conversation

cjcenizal
Copy link
Contributor

Addresses #11539

@cjcenizal cjcenizal added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label May 12, 2017
@cjcenizal
Copy link
Contributor Author

jenkins, test this

@thomasneirynck
Copy link
Contributor

hmm, CI seems to be on the fritz..

jenkins, test this

Copy link
Contributor

@thomasneirynck thomasneirynck left a comment

Choose a reason for hiding this comment

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

thx!

some small questions/comments.

I can't really tell when tabbing when I'm over the 'Play' and 'x'-buttons. Is this something that this PR should address as well, or is this a separate styling issue? fwiw, the orange outline makes it easy to see where you are tabbing. For example, here it is used on the expand-buttons.
image
Perhaps we can have something similar for play-and-x?

Tabbing over the play and x-buttons does not work when they are grayed-out. Is that supposed to be the case?

@cjcenizal
Copy link
Contributor Author

Thanks @thomasneirynck this is a really good point. We're going to improve visual states in a separate PRs. If you're interested, this is the issue that captures this phase of work: #11517.

CC @snide something to think about since you're working on the visual side of things right now.

Copy link
Member

@ppisljar ppisljar left a comment

Choose a reason for hiding this comment

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

LGTM

@cjcenizal cjcenizal merged commit dcc6b42 into elastic:master May 16, 2017
@cjcenizal cjcenizal deleted the 11539/visualize-sidebar-tabs-accessibility branch May 16, 2017 19:48
cjcenizal added a commit to cjcenizal/kibana that referenced this pull request May 16, 2017
* Make Visualize sidebar tabs keyboard accssible.
* Make Visualize Heatmap sidebar options keyboard accessible.
* Remove unnecessary kbn-accessible-click from Visualize reset button.
* Make Visualize Point Series sidebar options keyboard accessible.
cjcenizal added a commit that referenced this pull request May 16, 2017
* Make Visualize sidebar tabs keyboard accssible.
* Make Visualize Heatmap sidebar options keyboard accessible.
* Remove unnecessary kbn-accessible-click from Visualize reset button.
* Make Visualize Point Series sidebar options keyboard accessible.
snide pushed a commit to snide/kibana that referenced this pull request May 30, 2017
* Make Visualize sidebar tabs keyboard accssible.
* Make Visualize Heatmap sidebar options keyboard accessible.
* Remove unnecessary kbn-accessible-click from Visualize reset button.
* Make Visualize Point Series sidebar options keyboard accessible.
snide pushed a commit to snide/kibana that referenced this pull request Jun 2, 2017
* Make Visualize sidebar tabs keyboard accssible.
* Make Visualize Heatmap sidebar options keyboard accessible.
* Remove unnecessary kbn-accessible-click from Visualize reset button.
* Make Visualize Point Series sidebar options keyboard accessible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Visualizations Generic visualization features (in case no more specific feature label is available) Project:Accessibility release_note:enhancement v5.5.0 v6.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants