Skip to content
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

[Feature Request]: Move the progress bar to the bottom of the video player #5955

Closed
3 tasks done
chrrybmb64 opened this issue Oct 27, 2024 · 16 comments
Closed
3 tasks done
Labels

Comments

@chrrybmb64
Copy link

Guidelines

  • I have searched the issue tracker for open and closed issues that are similar to the feature request I want to file, without success.
  • I have searched the documentation for information that matches the description of the feature request I want to file, without success.
  • This issue contains only one feature request.

Problem Description

With Freetube's new video player the progress bar and video settings are now slightly above the bottom, making them more likely to obstruct parts of videos and also just not looking as nice.

freetubescreenshot

Proposed Solution

Move the progress bar and video settings all the way to the bottom, where they were with the old video player.

Alternatives Considered

n/a

Issue Labels

visual improvement

Additional Information

No response

@efb4f5ff-1298-471a-8973-3d47447115dc

This was the same with the old player and will not be changed in the new player

@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc closed this as not planned Won't fix, can't repro, duplicate, stale Oct 27, 2024
@chrrybmb64
Copy link
Author

This was the same with the old player

No, it's not the same as the old player. Maybe I should have phrased it better, I'm not just talking about the progress bar, I'm taking about the whole player menu. Here's a comparison.

freetube2

In the new player, the menu should be where the white box is, as it was in 0.21.3.

@absidue
Copy link
Member

absidue commented Oct 27, 2024

As @efb4f5ff-1298-471a-8973-3d47447115dc said, it's a new player so it looks slightly different and we won't be changing back to the old player.

@chrrybmb64
Copy link
Author

I'm not asking you to go back to the old player, I'm asking you to modify the new one. Is that not possible?

@absidue
Copy link
Member

absidue commented Oct 27, 2024

Theoretically it would be possible, but would likely break other parts of the UI (e.g. subtitles), make the player worse on touchscreens and mobile devices in the same way that the old one was. It would also be a major deviation from upstream, so resulting in more work on our side to update the modifications any time things change upstream, just a reminder that FreeTube had customized the previous player so much that it made it impossible to update to any newer version of video.js. So we definitely want to avoid doing unnecessary modifications, just because users refuse to get used to the new UI.

@chrrybmb64
Copy link
Author

chrrybmb64 commented Oct 27, 2024

Fair enough, thank you for the explanation

@jspast

This comment has been minimized.

@absidue
Copy link
Member

absidue commented Oct 27, 2024

It is not an issue and please don't waste the shaka-player developers time with this, as they definitely don't want to redesign their entire UI just because a few FreeTube users refuse to accept that things change over time.

@jspast
Copy link

jspast commented Oct 27, 2024

It is not an issue and please don't waste the shaka-player developers time with this, as they definitely don't want to redesign their entire UI just because a few FreeTube users refuse to accept that things change over time.

First, some users do see this as a problem, as was already expressed at least 3 times (and the beta is effectively less than 1 day old!). Second, I doubt it would require a redesign of the entire UI , but I have to agree this is only a minor design issue, and overall the new player experience is much better.

I want to make clear that your work on this was incredible, it is just sad that because of this really minor thing some users may want to keep using the older versions.

@kommunarr
Copy link
Collaborator

For discussion on how Shaka handles it, spacing appears to be ~25px taller than YT's on desktop view, which is not much in the span of things. You could argue that it should be a fixed height like YT instead of a dynamic padding-bottom: 2.5% to better accommodate above-average viewport sizes, and maybe change that base desktop number in doing so, but I don't think it is that big of a deal. It might be being remarked on a bit because Shaka does not have the icons fill the height of the allocated container space (which I would consider to be a minor visual area for improvement), and it makes it stand out more visually compared to the previous video player.

@suy
Copy link

suy commented Oct 31, 2024

Let me please start my comment clarifying that it's totally understandable if Freetube can't implement this on its own due to library limitations. I'm more an app developer than a library developer, and my understanding is that the project is a lot of unpaid work as a volunteer, so I'm well aware of time/interest limitations (and I appreciate the whole project existing).

It is not an issue and please don't waste the shaka-player developers time with this, as they definitely don't want to redesign their entire UI just because a few FreeTube users refuse to accept that things change over time.

This, however, and with all due respect, I'm pretty confident that is a wrong point of view. I'm not used to the old UI because I never used it. I've always used Freetube the current way, and the controls obscuring the playback has been problematic a few times to me, without knowing how it was in the past.

Today I've searched the issues because of #5951 (the big play button) was just too obtrusive for me. I've also found this issue because someone has found the controls obtrusive in some way, and I think it's not a matter of taste: they are objectively blocking more video area than before.

I've filled #6050 to ask for the possibility of the controls be less in the way in the cases where it might matter, but if that's not possible to do, I don't see why it would not be possible to politely ask for a feature request on the player library if that can reduce the surface area that gets blocked meanwhile. Perhaps it doesn't happen at all (again, volunteer work, lack of resources, etc.), but I don't see the reasoning for discouraging users to try to solve an objective issue by reporting this on the right place.

Thank you.

@absidue
Copy link
Member

absidue commented Oct 31, 2024

The only people in the world that think the shaka-player UI is bad are FreeTube users, so please don't flood shaka-players issue tracker with requests like this.

@jspast
Copy link

jspast commented Oct 31, 2024

For discussion on how Shaka handles it, spacing appears to be ~25px taller than YT's on desktop view, which is not much in the span of things. You could argue that it should be a fixed height like YT instead of a dynamic padding-bottom: 2.5% to better accommodate above-average viewport sizes, and maybe change that base desktop number in doing so, but I don't think it is that big of a deal. It might be being remarked on a bit because Shaka does not have the icons fill the height of the allocated container space (which I would consider to be a minor visual area for improvement), and it makes it stand out more visually compared to the previous video player.

Thanks so much for your input here. Following your findings, I was able to experiment with the CSS on DevTools and find exactly what we could change to make the bar fit better. This is what shaka-player does by default:

image

Modifying it to 'padding: 0 0 10px', where 10 is just an arbitrary number to test, makes the player behaves as we normally expect it should, given it is what YouTube and basically every video focused platform does. With the little testing I did, nothing seems to break, but the position of the settings menus do change relative to the bar position.

Next, I would like to research what was shaka-player reasoning behind this choice to better understand what could be done.

@absidue
Copy link
Member

absidue commented Oct 31, 2024

Next, I would like to research what was shaka-player reasoning behind this choice to better understand what could be done.

As shaka-player is made by Google they based it on the default Chromium player UI and designed it to work on desktop, mobile devices and TVs.

@jspast
Copy link

jspast commented Oct 31, 2024

Next, I would like to research what was shaka-player reasoning behind this choice to better understand what could be done.

As shaka-player is made by Google they based it on the default Chromium player UI and designed it to work on desktop, mobile devices and TVs.

So I did some research and you are correct. The thing is: I couldn't reproduce the same behavior on the current version Chromium player UI, so maybe shaka-player default UI is actually outdated and all of this could be "fixed" upstream.

However, I also went through their official documentation where they mention some UI themes and a YouTube theme caught my attention. Look what it does:

image
Moving bottom control to bottom

Finally, I was able to implement this on FreeTube with just 2 lines (also changed the width to match). It's important to say that the buttons clickable area do not change, so it should not affect accessibility other than the fact that the buttons are overall a bit closer to the screen border on full-screen.

Before
image
After
image

I could open a PR if this sounds good.

The only remaining question is what to do with the settings menus. By default, they are 30px from the bottom, so their position is not even consistent with the bar's default 2.5% padding. There is already an issue somewhat related to that: #6035.

Sorry for such a long comment.

@jspast
Copy link

jspast commented Dec 9, 2024

Update: this has been properly implemented and merged with #6007. Thanks FreeTube devs!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

6 participants