Skip to content

Conversation

@Xurdejl
Copy link
Contributor

@Xurdejl Xurdejl commented Feb 1, 2026

Add a new alternative layout called modern-compact.

closes #425

preview

@Samillion
Copy link
Owner

Samillion commented Feb 1, 2026

That is a beautiful compact layout, seriously well done.

Seekbar position is a bit different for me than shown in your preview, not sure why. Is it an issue on my end?

refY - 82 (current)
image
refY - 72 (adjusted)
image

@Samillion
Copy link
Owner

Also, as I'm sure he'll love to see this, I have to CC @Keith94

@Xurdejl
Copy link
Contributor Author

Xurdejl commented Feb 1, 2026

That is a beautiful compact layout, seriously well done.

Seekbar position is a bit different for me than shown in your preview, not sure why.

refY - 82:

image and this is with `refY - 72`: image Is it an issue on my end?

Yeah, that was my mistake. It was supposed to be refY - 72, I was testing changes across two different files in my config folder and the cloned repo folder and might have undone something while editing the files.

@Samillion
Copy link
Owner

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.

@Xurdejl Xurdejl force-pushed the feat/compact-layout branch from 17584ed to ecb235d Compare February 1, 2026 23:15
@Xurdejl
Copy link
Contributor Author

Xurdejl commented Feb 1, 2026

okay, now should be fine now. I always find it a bit of a headache to rebase to keep the commits tidy

@Keith94
Copy link
Contributor

Keith94 commented Feb 1, 2026

Thanks for adding that so fast! Much nicer overall.

I did a quick check and noticed a few things compared to the modern layout.

  • The starting and ending position of the buttons is more "pushed in" towards the middle.
  • Volume bar is noticeably thinner (vertically).
  • Chapter title is always shown even if turned off.
  • Audio and subtitle positions are swapped around.
  • Chapter skip, jump, info buttons are always hidden. Maybe that's expected, or can be they visible on the right side?

@Samillion
Copy link
Owner

Samillion commented Feb 1, 2026

  • Chapter skip, jump, info buttons are always hidden. Maybe that's expected, or can be they visible on the right side?

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.

@Xurdejl
Copy link
Contributor Author

Xurdejl commented Feb 1, 2026

  • The starting and ending position of the buttons is more "pushed in" towards the middle.
  • Volume bar is noticeably thinner (vertically).

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.

  • Chapter title is always shown even if turned off.

I completely forgot there was a user option for that. Will add it now.

  • Audio and subtitle positions are swapped around.

I can also change that.

  • Chapter skip, jump, info buttons are always hidden. Maybe that's expected, or can be they visible on the right side?

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

@Samillion
Copy link
Owner

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.

@Keith94
Copy link
Contributor

Keith94 commented Feb 2, 2026

The starting and ending position of the buttons is more "pushed in" towards the middle.
Volume bar is noticeably thinner (vertically).
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 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.

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

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

@Samillion
Copy link
Owner

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.

@Samillion
Copy link
Owner

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.

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.

@Keith94
Copy link
Contributor

Keith94 commented Feb 2, 2026

Indented start_x is good by me IMO.
I also feel that the volume bar width should be reduced to match something like YouTube's. It takes up a lot of real estate especially for a compact mode.

@Xurdejl Xurdejl force-pushed the feat/compact-layout branch from ecb235d to 32b480a Compare February 2, 2026 00:19
@Xurdejl
Copy link
Contributor Author

Xurdejl commented Feb 2, 2026

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

@Keith94
Copy link
Contributor

Keith94 commented Feb 2, 2026

Thanks for the fixes. The script should probably check whether user toggled these ones off, too

playlist_button=no
subtitles_button=no
ontop_button=no
fullscreen_button=no

@Keith94
Copy link
Contributor

Keith94 commented Feb 2, 2026

This is not urgent either, but supporting vertical videos would need some tweaking in the future

@Xurdejl Xurdejl force-pushed the feat/compact-layout branch from 32b480a to 0050923 Compare February 2, 2026 00:42
@Xurdejl
Copy link
Contributor Author

Xurdejl commented Feb 2, 2026

This is not urgent either, but supporting vertical videos would need some tweaking in the future

I'll need help with this because I don't know the correct way to do it

@Samillion
Copy link
Owner

Thanks again for all the effort. I will merge, and start working on any additions//adjustments in the next couple of days.

@Samillion Samillion merged commit d917f6e into Samillion:main Feb 2, 2026
@Xurdejl Xurdejl deleted the feat/compact-layout branch February 2, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Better default layout

3 participants