-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Make sure we have sufficient space for the bar as well if content is tiny #3898
Conversation
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.
It looks like some tests need to be adjusted
Sorry about that, was away from the machine a couple days. Fixed now |
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.
This PR does unfortunately seem to introduce various bugs with the showing and hiding of tabs. One thing I noticed is that the window sometimes becomes larger when selecting a different tab if there are tabs in the dropdown menu:
Screencast.from.2023-05-22.20-31-36.webm
I also noticed that you can't make the window smaller again if you make it large enough to show more tabs:
Screencast.from.2023-05-22.20-33-40.webm
Thanks for catching that, should be fixed now |
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.
I still can't seem to make the window smaller if new tabs are shown.
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.
Seems fine now. Thanks 👍
Fixes #3885
Checklist: