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

#959 add a max-height and center element with alignItems: center #961

Merged
merged 4 commits into from
Apr 28, 2017
Merged

#959 add a max-height and center element with alignItems: center #961

merged 4 commits into from
Apr 28, 2017

Conversation

wcastand
Copy link
Contributor

@wcastand wcastand commented Apr 26, 2017

Issue: #959

I'm not sure i did the things right (never used lerna so i was a bit lost ). Let me know if i missed something.

What I did

I added some style to the tabtitles to center his children.

How to test

In the example, add a react component as a Title with an button for example and some padding or margin.

@usulpro usulpro self-assigned this Apr 27, 2017
@usulpro
Copy link
Member

usulpro commented Apr 27, 2017

@wcastand thanks for PR.

Consider applying maxHeight and overflow to the tablink style

@wcastand
Copy link
Contributor Author

wcastand commented Apr 27, 2017

I put max-Height in the tablink style.
I didn't know how to test it on the right panel in the example of storybook-ui, so let me know if i miss something again.

Thanks for the help on this :)

We should update the doc too but i can't find it and my english is far from perfect for a documentation :/

justifyContent: 'flex-start',
alignItems: 'center',
maxHeight: 60,
overflow: 'hidden',
Copy link
Member

@usulpro usulpro Apr 27, 2017

Choose a reason for hiding this comment

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

just remove it from here:

maxHeight: 60,
overflow: 'hidden',

Copy link
Member

@usulpro usulpro left a comment

Choose a reason for hiding this comment

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

remove

maxHeight: 60,
overflow: 'hidden',

from tabbar

@wcastand
Copy link
Contributor Author

I think we're good now :)

Copy link
Member

@ndelangen ndelangen left a comment

Choose a reason for hiding this comment

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

Looks good to me, tested locally

@ndelangen ndelangen merged commit 57b7161 into storybookjs:master Apr 28, 2017
@shilman shilman added the misc label May 27, 2017
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.

4 participants