-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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: extra arrow in tab component on resize #16652
fix: extra arrow in tab component on resize #16652
Conversation
✅ Deploy Preview for v11-carbon-react ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-elements ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still seeing some issues in chrome with the left arrow. It's not overlaying the tabs properly and doesn't seem to be clickable in certain situations. Selecting the tabs themselves does cause it to update the scroll position but it doesn't appear the button is actually being clicked
CleanShot.2024-06-05.at.13.30.28-converted.mp4
Thanks Taylor, I didn't checked that functionality bcs the issue was related to normal screen and arrow should not be visible. Let me check the functionality part. |
fixed the left arrow issue although I feel sometimes when we play with arrows, it remove the arrows bcs of some calculations when all tabs are visible (not fully), I will raise a separate issue for this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks to be working correctly now, I did tag design for a visual review also.
Just needs a visual review @alina-jacob |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @preetibansalui, nice work! 🔥
Just wanted to confirm one small thing with you: Is it okay if the arrow still shows up while I'm increasing the screen size (you can see that at 00:05 sec)? It disappears when the mouse is moved out, but I wanted to check if this is acceptable. Everything else looks good to me from a design perspective!
Screen.Recording.2024-06-10.at.4.17.08.PM.mov
Hi Kritvi, It should remove the arrow on a particular width, Here we have a debounce value before we apply changes on window's scroll event, Because of that it remove the arrow after you are done with resizing. I believe it should be acceptable bcs a user should not change window's width like this. |
Hi @preetibansalui, this looks neat! 🚀 Just noticed this one change between the current storybook and the preview that you've shared (please ignore if irrelevant) Sharing a screenshot for reference, left is current storybook and right is a screenshot from the preview link shared above: |
Hi Alina, It looks like an issue, let me look into it. Thanks for pointing this out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ed.mp4
Thanks Taylor, I didn't checked that functionality bcs the issue was related to normal screen and arrow should not be visible. Let me check the functionality part.
In that case, it looks good to me after addressing the issue Alina mentioned!
Hi @alina-jacob, I have made the changes, can you please review it again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to go @preetibansalui! 🚀 💯
7e57d8e
Closes #16484
Fix: Extra arrow is visible in Tab component on resize
Testing / Reviewing