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

feat(ComponentExample): add show html button to docs #547

Merged
merged 1 commit into from
Sep 25, 2016

Conversation

levithomason
Copy link
Member

@levithomason levithomason commented Sep 25, 2016

It is very helpful to see the rendered HTML output for examples. I find I am constantly drilling through the dev tools to very the output of example markup.

This PR adds an HTML5 logo icon button next to the code button for all examples. You can now toggle the JSX and/or the HTML into view:

image

@levithomason levithomason changed the title docs(ComponentExample): add show html button feat(ComponentExample): add show html button Sep 25, 2016
@levithomason levithomason changed the title feat(ComponentExample): add show html button feat(ComponentExample): add show html button to docs Sep 25, 2016
@codecov-io
Copy link

codecov-io commented Sep 25, 2016

Current coverage is 98.86% (diff: 100%)

Merging #547 into master will not change coverage

@@             master       #547   diff @@
==========================================
  Files           108        108          
  Lines          1764       1764          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1744       1744          
  Misses           20         20          
  Partials          0          0          

Powered by Codecov. Last update 147a70d...8096a15

@levithomason levithomason merged commit 22484d4 into master Sep 25, 2016
@levithomason levithomason deleted the docs/show-html branch September 25, 2016 15:20
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