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

addon-info: make the info overlay be fixed #914

Merged
merged 3 commits into from
Apr 28, 2017
Merged

addon-info: make the info overlay be fixed #914

merged 3 commits into from
Apr 28, 2017

Conversation

Kiwka
Copy link
Contributor

@Kiwka Kiwka commented Apr 18, 2017

This fix will make the view of the information be not affected by the decorators positioning, that can be applied to the story.

Can be the solution for this issues:
storybook-eol/react-storybook-addon-info#137
storybook-eol/react-storybook-addon-info#40

@codecov
Copy link

codecov bot commented Apr 28, 2017

Codecov Report

Merging #914 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #914   +/-   ##
=======================================
  Coverage   26.93%   26.93%           
=======================================
  Files         192      192           
  Lines        4384     4384           
  Branches      705      705           
=======================================
  Hits         1181     1181           
  Misses       3203     3203
Impacted Files Coverage Δ
packages/addon-info/src/components/Story.js 22.89% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9de2b21...e5ca5b6. Read the comment docs.

@ndelangen ndelangen merged commit b720043 into storybookjs:master Apr 28, 2017
@ndelangen
Copy link
Member

Thank you @Kiwka !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants