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(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. #441

Merged
merged 5 commits into from
Mar 14, 2022

Conversation

creador-dev
Copy link
Member

No description provided.

@creador-dev creador-dev added the bug Something isn't working label Jan 7, 2022
@creador-dev creador-dev changed the base branch from development to version/2.12.5 February 14, 2022 13:31
@iamleigh iamleigh changed the title 🐛 fix(tabs): fixed overlap issue on first and last element fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. Mar 4, 2022
@iamleigh iamleigh self-requested a review March 4, 2022 00:11
Copy link
Contributor

@iamleigh iamleigh left a comment

Choose a reason for hiding this comment

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

@creador-dev there's a bug. Try adding tabs with long names and notice right arrow to start navigating doesn't appear. To show it up, you need to scroll to right first.

Copy link
Contributor

@iamleigh iamleigh left a comment

Choose a reason for hiding this comment

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

@creador-dev there's a bug. As shown in the screenshot below, try adding tabs with long names and notice right arrow to start navigating doesn't appear. To show it up, you need to scroll to right first.

@creador-dev creador-dev changed the title fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. fix(tabs): overflow flushed tabs menu doesn't have enough space to place navigation buttons. Mar 4, 2022
@creador-dev creador-dev requested a review from iamleigh March 7, 2022 13:23
@iamleigh iamleigh changed the title fix(tabs): overflow flushed tabs menu doesn't have enough space to place navigation buttons. fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. Mar 9, 2022
Copy link
Contributor

@iamleigh iamleigh left a comment

Choose a reason for hiding this comment

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

@creador-dev review again your changes. There is something that's causing an alignment issue in the tab buttons label. Also, they're not overflowing correctly.

Screen Shot 2022-03-08 at 22 16 21

@creador-dev creador-dev changed the title fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. 🐛 fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. Mar 10, 2022
@iamleigh iamleigh changed the title 🐛 fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. fix(tabs): Overflow flushed tabs menu doesn't have enough space to place navigation buttons. Mar 13, 2022
@iamleigh iamleigh merged commit 50b30a4 into version/2.12.5 Mar 14, 2022
@iamleigh iamleigh deleted the fix/SUI-282 branch March 14, 2022 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants