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
Unify window style #380
Unify window style #380
Conversation
Some funky :not being used to resolve .tb-window-tabs from .tb-popup-tabs, but in a moment I'll be unifying these two into one normal block of code and it'll all be good 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.
Went over all the changes, it seems to make sense. I still have to visually look it over by checking out the PR
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.
Oddly enough personal notes doesn't seem to want to show on a regular frontpage (it is there hiding at the bottom) but does show in queue. Not sure what is going on there but it is specific to this PR.
Other than that it looks good to me.
Latest commit fixes that issue for me. |
This PR unifies the
.tb-window-*
,.tb-popup-*
, and.tb-notification-*
classes under a single.tb-window-*
scheme including modifier classes. Also adds an argument toTBui.overlay
to control the tab bar orientation, to make the profile overlay less of a special case, and adds documentation to the arguments of that function. The overlay CSS was also modified to not rely on containing a specific type of window in case that's desirable in the future (thoughTBui.overlay
still generates large static windows and nothing else).TBui.popup()
returns a.tb-window.tb-window-draggable
TBui.overlay()
uses a.tb-window.tb-window-large.tb-window-vertical-tabs
TBui.showNotification()
uses a basic.tb-window
Technically, there are still slight differences in the markup generated by
TBui.popup
andTBui.overlay
, but those differences are mostly in how data for multiple tabs is stored. This PR is only concerned with unifying the classes controlling visual appearance; a future PR can handle unifying the internal structure, since that will probably require breaking out a dedicated tabber component and retrofitting it into the existing UI generators.Supercedes #327. Fixes #240.