-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: button bar decluttering options (playlist & tooltips) #134
Conversation
This was much needed.. thanks @Commander07 |
Brilliant, yet another great addition. Thank you so much. I hate to ask, but there was a recent PR that renamed some of the osc styles, which introduced a conflict. Would you mind updating this, please? #135 At a glance I think it's just:
|
Appologizes on that, I thought I could sneak that in without causing any conflicts since there wasn't any use of styles from what I saw. |
Not at all, this was definitely my fault. I should have checked before merging. I would fix this myself, but right now I only have my phone. Any inconvenience is on me, sorry for that. |
Think it should be resolved now. |
Much appreciated, indeed it is resolved. |
This PR adds button bar decluttering options for the playlist button and tooltips.
Changes
hide_empty_playlist_button
which hides the playlist button when no playlist exists.gray_empty_playlist_button
which grays out the playlist button when no playlist exists. Which should solve Gray out playlist button if there's no playlist? #129.tooltips_for_disabled_elements
which disables tooltips for disabled/grayed out buttons. E.g. the sub track button when no subtitles are available.tooltip_hints
which toggles the clarification tooltips of the information, loop, ontop and screenshot buttons.