Skip to content

fix: remove rounded corners for Shorts thumbnails and video previews - #4258

Open
chungus-big wants to merge 1 commit into
code-charity:masterfrom
chungus-big:shorts-and-preview
Open

fix: remove rounded corners for Shorts thumbnails and video previews#4258
chungus-big wants to merge 1 commit into
code-charity:masterfrom
chungus-big:shorts-and-preview

Conversation

@chungus-big

Copy link
Copy Markdown
Contributor

Summary

  • Shorts Thumbnails: Removed rounded corners for Shorts thumbnails using .shortsLockupViewModelHostThumbnailParentContainer.
  • Hover Video Preview: Fixed static rounded corners during hover video playback by adding ytd-video-preview #media-container.

Question for Reviewers

  • Should we also remove rounded corners for the hover shadow effect and the main watch page video player? I left these unchanged for now as I wasn't sure if that behavior should be included in this setting.

Testing

  • Updated unit tests in tests/unit/squared-thumbnails.test.js to assert the new selectors.
  • Verified all unit tests via npx jest (all 21 test suites passed).

…ontainer

Removed rounded corners for Shorts thumbnails using .shortsLockupViewModelHostThumbnailParentContainer. Fixed an issue where video preview corners remained rounded during hover playback using ytd-video-preview #media-container. Hover shadow and watch page player were left unchanged pending feedback.
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.

1 participant