Skip to content

fix: apply thumbnail size setting to playlist video thumbnails in search results (Fixes #4248) [fj4WqyCCw3C5ShR1RfB7MoBPTpkRrBFYP1uT35g3MvT] - #4251

Open
waterWang wants to merge 1 commit into
code-charity:masterfrom
waterWang:fix/playlist-thumbnail-size-4248
Open

fix: apply thumbnail size setting to playlist video thumbnails in search results (Fixes #4248) [fj4WqyCCw3C5ShR1RfB7MoBPTpkRrBFYP1uT35g3MvT]#4251
waterWang wants to merge 1 commit into
code-charity:masterfrom
waterWang:fix/playlist-thumbnail-size-4248

Conversation

@waterWang

Copy link
Copy Markdown

What this PR fixes

When the user sets a custom thumbnail size (Large / Medium / Small / x-small / xx-small) in ImprovedTube settings, the CSS only applied to ytd-video-renderer (regular search results) and ytd-compact-video-renderer (sidebar). Playlist video results in search use ytd-playlist-video-renderer, which was not targeted, so playlist thumbnails always remained at the default YouTube size.

Change

Added 5 new CSS rules in js&css/extension/www.youtube.com/styles.css to apply the same max-width / min-width values to ytd-playlist-video-renderer ytd-thumbnail for each thumbnail size setting.

How to test

  1. Install/load the extension from this branch
  2. Go to YouTube search and search for something that returns both video and playlist results
  3. Set Thumbnail Size to Small (or any non-default value)
  4. Before: playlist video thumbnails stay at default size
  5. After: playlist video thumbnails match the selected size

Fixes #4248

…rch results (Fixes code-charity#4248)

When the user sets a custom thumbnail size (Large / Medium / Small / x-small / xx-small),
the CSS only applied to ytd-video-renderer (regular search results) and
ytd-compact-video-renderer (sidebar). Playlist video results in search
use ytd-playlist-video-renderer, which was not targeted, so playlist
thumbnails always remained at the default YouTube size.

This commit adds the same size rules for ytd-playlist-video-renderer ytd-thumbnail.
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.

🐞Playlist Thumbnails aren't registered as Thumbnails

1 participant