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

fix: Add html lang attribute to iframe #7892

Merged
merged 1 commit into from
Aug 30, 2019

Conversation

resource11
Copy link
Contributor

Issue: related to Issue #2538 and PR #3065

What I did

  • Now that Storybook is refactored to use Typescript, the html element's lang attribute is missing in the core index.ejs template. This PR adds that attribute back in. This fix will improve the accessibility of the app (html elements need a lang attribute), and it will ensure components that use the CSS . hyphens property will display hyphenated text as intended. If you're curious, here's some links for learning more about why pages need the lang attribute and why the lang attribute is needed for CSS hyphenation.

How to test

  • Is this testable with Jest or Chromatic screenshots? No
  • Does this need a new example in the kitchen sink apps? No
  • Does this need an update to the documentation? No

If your answer is yes to any of these, please make sure to include it in your PR.

@vercel
Copy link

vercel bot commented Aug 28, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

@ndelangen ndelangen added this to the 5.2.0 milestone Aug 30, 2019
@ndelangen ndelangen self-assigned this Aug 30, 2019
@ndelangen ndelangen merged commit 6d04730 into storybookjs:next Aug 30, 2019
@resource11 resource11 deleted the hotfix/add-iframe-lang-attr branch August 31, 2019 04:34
@resource11 resource11 restored the hotfix/add-iframe-lang-attr branch April 14, 2020 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants