-
Notifications
You must be signed in to change notification settings - Fork 26
feat: add modern-compact layout #435
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
Conversation
|
Also, as I'm sure he'll love to see this, I have to CC @Keith94 |
|
Ah, I've done that many times 😅. I genuinely love this layout, pretty sure it'll be my default. Let me know whenever it's ready to merge. Take all the time you need. |
17584ed to
ecb235d
Compare
|
okay, now should be fine now. I always find it a bit of a headache to rebase to keep the commits tidy |
|
Thanks for adding that so fast! Much nicer overall. I did a quick check and noticed a few things compared to the modern layout.
|
Expected, definitely. As this is meant to be compact. Menu and context menu give access to all the extra stuff. We can always adjust any additions in the future if needed. Consider this a base start for the layout. |
Both of these are used in other player ui's because of visual hierarchy. I can make them like they were before if that is preferred.
I completely forgot there was a user option for that. Will add it now.
I can also change that.
Those are not even added since the idea was to use the menu button (or hotkeys) in case you want to access the less frequent actions |
Indeed. I think in the future the most requested will be chapter skip and jump buttons. Anything else is as conveniently accessed through menu/context menu. Nevertheless, it's definitely not urgent. |
I see what you mean, and it's started to grow on me. These new values could be applied to the other modern and modern-image layouts too, no? At some point anyway.
This is OK since I like a cleaner look anyway. Might be worth explaining the differences (ie button omissions) between the layouts somewhere in the docs etc |
Agreed. I'll definitely make a big section about this layout and its details in the next release as well. |
The volume bar is too thin, I agree to that. The starting position can be whatever is decided to be honest. You guys have a much better grasp on UI hierarchy than I do. |
|
Indented start_x is good by me IMO. |
ecb235d to
32b480a
Compare
|
Made the volume bar shorter and thicker, swapped the position of the audio and subtitle buttons, and fixed the chapter title not hiding when the user option was set to false |
|
Thanks for the fixes. The script should probably check whether user toggled these ones off, too playlist_button=no |
|
This is not urgent either, but supporting vertical videos would need some tweaking in the future |
32b480a to
0050923
Compare
I'll need help with this because I don't know the correct way to do it |
|
Thanks again for all the effort. I will merge, and start working on any additions//adjustments in the next couple of days. |




Add a new alternative layout called modern-compact.
closes #425