Playlist Size Too Small

[jpeg9999]

⚬ PROBLEM:
Love the extension BTW. When ImproveYoutube is activated, the playlist only takes up about 1/2 the sidebar, but does not behave this way without extension active. With regular Youtube, the playlist is the same height as the video--this is what users are accustomed to--especially us who use the playlist a lot.

I see there are options within the extension to hide or collapse related videos, but this does not increase the size of playlist.

⚬ SOLUTION:
It would be wonderful to have a sidebar option so the playlist can be normal size (same height as video).

//
Thank you!

[DarshanDixit05]

Hey, I want to work on this issue. Can you please assign it to me?

[jpeg9999]

Hey, I want to work on this issue. Can you please assign it to me?

How does it get assigned?

[DarshanDixit05]

The one with the write access to the repo, anyways I'm working on it locally.

[jpeg9999]

The one with the write access to the repo, anyways I'm working on it locally.

sweet

[D-Rekk]

Hello! First of all I'm culprit thanks for noticing this feature/bug🥲

Why is my playlist so smol?😧

What happened is that, while I was working on this awesome Sidebar feature, I noticed that the playlist height was quirky with the extension on:

• Changing player size makes playlists lose their scrollbars #1693, infinite playlist height.
• For the Comments -> Sidebar feature I needed to apply JavaScript to resize the video player, and with a window resize the playlist would sometimes disappear, possibly some collision with YouTube native JavaScript.

I could've applied more JavaScript, but I only style with JS if strictly necessary. Eventually what I did was to apply a fixed height as global CSS:
html[data-page-type=video] #playlist #items { max-height: calc(70px * 4 + 4px) ;
padding-bottom: 0;} /* MIGHT FIX #1693, currently shows up to 4 videos */

As you saw the playlist shows a max 4 videos right now.

How come?

Honestly this was chosen specifically for the Comments-Sidebar feature because comments and playlist would share the same container so a small number brought many benefits:

• Playlist wouldn't fill most of the viewport height
• Comments would start being fetched
• I found aesthetically pleasing to have a fixed integer rather than having 7,2 or 5,4 videos showing as you resize the window.

What can we do to improve?

• Change the style to be applied only on Comments -> Sidebar enabled, but probably the no-scrollbar bug will reappear
• Convert the fixed number of videos to a CSS variable and be able to set that number in the settings (a new feature✨)
• Resize Playlist height with JavaScript, bringing the original behavior
• Other you suggest
• Nothing, it's fine?

Let me know what you think

[jpeg9999]

is there anything i can add in Settings/Developer Options/CSS or Javascript to override the fixed height?

well i changed 4 to 8 and loaded unpacked. i feel like most people want the playlist to be normal size.

[ImprovedTube]

hi guys & thanks

assignment

thanks @DarshanDixit05 chances that two people work on it at once are still low. I can invite you guys to the team so you can also assign & tag/label

[ImprovedTube]

{ max-height: calc(70px * 4 + 4px) padding-bottom: 0;}

• closer to the original:
  {max-height: min(var(--it-player-height), 79vh)
{max-height: min(calc(var(--it-player-height) - 12px), 79vh)
  • didnt test, but to fix the same for the transcript-box "max-height:77vh" helped: https://github.com/code-charity/youtube/blob/f520502f8c23fee1ec1cc9d3428e613efa7e0a2c/js%26css/extension/www.youtube.com/appearance/sidebar/sidebar.css#L34C1-L35C34

&

• we can avoid running anything with no settings.
  • so this can depend player size set or comments sidebar being set:
    html[data-page-type=video][it-comments-sidebar='true'] #playlist #items,
html[data-page-type=video]:not([it-player-size]) #playlist #items,
html[data-page-type=video]:not([it-player-size='do_not_change']) #playlist #items
{ max-height: ... }
  • same for the JS resizing by @D-Rekk

[D-Rekk]

Since this is applying only on [it-comments-sidebar='true'] I'm closing this issue.