Skip to content

fix: menu text-overflow#989

Merged
tsi merged 1 commit intomasterfrom
VIDEO-20647-menu-overflow
Feb 26, 2026
Merged

fix: menu text-overflow#989
tsi merged 1 commit intomasterfrom
VIDEO-20647-menu-overflow

Conversation

@tsi
Copy link
Collaborator

@tsi tsi commented Feb 24, 2026

Long chapter names in the chapters menu (and similar menus like subtitles) caused the menu to grow and move off screen because there was no width limit or text truncation.

Solution

  • Remove custom styles messing with the menu width
  • Add CSS ellipsis to menu-items

@tsi tsi requested a review from a team as a code owner February 24, 2026 06:10
@netlify
Copy link

netlify bot commented Feb 24, 2026

Deploy Preview for cld-video-player ready!

Name Link
🔨 Latest commit d49eaa3
🔍 Latest deploy log https://app.netlify.com/projects/cld-video-player/deploys/699d40cf18d02000084b1e85
😎 Deploy Preview https://deploy-preview-989--cld-video-player.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Feb 24, 2026

Deploy Preview for cld-vp-esm-pages ready!

Name Link
🔨 Latest commit d49eaa3
🔍 Latest deploy log https://app.netlify.com/projects/cld-vp-esm-pages/deploys/699d40cf724fb30008cf511a
😎 Deploy Preview https://deploy-preview-989--cld-vp-esm-pages.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@tsi tsi self-assigned this Feb 24, 2026
@tsi tsi merged commit 8816699 into master Feb 26, 2026
10 checks passed
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.

3 participants