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

[Tab] Allow centered tab menu #45

Merged
merged 3 commits into from
Aug 10, 2018
Merged

[Tab] Allow centered tab menu #45

merged 3 commits into from
Aug 10, 2018

Conversation

prudho
Copy link
Contributor

@prudho prudho commented Jun 29, 2018

This PR allow tab menu to be centered. Also works with the secondary style. Closes Semantic-Org/Semantic-UI#5589.

Thanks @franckpopchef for the implementation idea !

@prudho prudho added type/feat Any feature requests or improvements lang/css Anything involving CSS state/awaiting-reviews Pull requests which are waiting for reviews state/awaiting-docs Pull requests which need doc changes/additions labels Jun 29, 2018
y0hami
y0hami previously approved these changes Jul 2, 2018
@y0hami y0hami added Can Merge and removed state/awaiting-reviews Pull requests which are waiting for reviews labels Jul 2, 2018
@ColinFrick ColinFrick added the state/on-hold Issues and pull requests which are on hold for any reason label Jul 4, 2018
@y0hami y0hami added state/awaiting-reviews Pull requests which are waiting for reviews and removed Can Merge labels Jul 4, 2018
@y0hami y0hami changed the base branch from next to beta July 4, 2018 10:05
@y0hami y0hami requested a review from ColinFrick July 4, 2018 13:18
@y0hami y0hami assigned y0hami and ColinFrick and unassigned y0hami Jul 27, 2018
This corresponds to the right modifier
Caveat: Does not display a border left or right, because :after definition is used for pointing menu

Closes Semantic-Org/Semantic-UI#5589
@ColinFrick ColinFrick requested a review from y0hami August 9, 2018 09:48
@ColinFrick ColinFrick removed the state/on-hold Issues and pull requests which are on hold for any reason label Aug 9, 2018
@romaninsh
Copy link

shouldn't it be "centered menu" or "center aligned menu", not "center menu" ?

no sample or doc.

@ColinFrick
Copy link
Member

ColinFrick commented Aug 10, 2018

@romaninsh it follows the same syntax as the alignment modifiers for the menu (left / right)

Docs will be updated before the release.

@y0hami y0hami removed the state/awaiting-reviews Pull requests which are waiting for reviews label Aug 10, 2018
@y0hami y0hami merged commit a0af175 into fomantic:beta Aug 10, 2018
@y0hami y0hami added state/has-docs A issue/PR which requires documentation changes and has the corresponding PR open in the docs repo and removed state/awaiting-docs Pull requests which need doc changes/additions labels Sep 2, 2018
@prudho prudho deleted the fix-5589 branch April 11, 2019 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/css Anything involving CSS state/has-docs A issue/PR which requires documentation changes and has the corresponding PR open in the docs repo type/feat Any feature requests or improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants