-
Notifications
You must be signed in to change notification settings - Fork 102
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
toolbar height is to small #110
Comments
I have created a pull request with the 1st proposal in place. |
Thanks for reporting this @soundslikeodd ! I gave a try to #111 but I must say I find this large blank a bit of a waste of space when you only have one row of tabs. Did you already look into what solution 2 involves ? Thanks ! |
Yeah, I'd say either adjust the toolbar height dynamically, or we do a tab-completable dropdown with the full list, and keep the most frequently used workspaces in the top list. |
I agree that solution 2 is the better solution. I am not the best at front end dev work so the 1st solution was the quickest for me to spin up. I can come up with a sample for the 2nd solution but it might take me a bit. I like the tabs over a dropdown because it is quicker to switch between workspaces. Putting the most frequently used on top is a good way to mitigate this issue but it would still take some time to find a workspace if there are a large number of them. |
I have a rough implementation for the 2nd solution. |
Closed by #112, thanks ! For the record, this fix will be part of upcoming version 0.2.13. |
Currently the height of the toolbar div is to small. If there are many tabs they wrap and are hidden by the view div. Both the toolbar and the view have absolute positions.
There are two possible solutions.
The text was updated successfully, but these errors were encountered: