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

Update heading hierarchy for instructions #41970

Merged
merged 2 commits into from
Jul 29, 2019

Conversation

myasonik
Copy link
Contributor

Summary

Updates the heading hierarchy: the page had no h1, but several h2s at different levels so I promoted one.

This PR was going to be bigger but the other a11y issues turned out to be in EUI.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials

For maintainers

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@elasticmachine
Copy link
Contributor

💔 Build Failed

@myasonik myasonik force-pushed the fix/instructions-heading-hierarchy branch from a39f7cc to ca3d935 Compare July 26, 2019 16:48
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Member

@markov00 markov00 left a comment

Choose a reason for hiding this comment

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

Code LGTM

@myasonik myasonik merged commit 5aea728 into elastic:master Jul 29, 2019
@myasonik myasonik deleted the fix/instructions-heading-hierarchy branch July 29, 2019 11:55
myasonik pushed a commit to myasonik/kibana that referenced this pull request Jul 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Project:Accessibility release_note:fix Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.4.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants