From 8de33b7cb14530dbca3608ac98f4223e402ef3dc Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Sun, 25 Jan 2026 02:16:32 +0000 Subject: [PATCH] fix(ui): remove display:none from #playerModal selector to fix modal visibility The #playerModal ID selector in `css/tailwind.source.css` incorrectly set `display: none`. This ID selector has higher specificity than the `.hidden` utility class (applied by Tailwind) used by `VideoModal.js` to toggle visibility. When `VideoModal.js` removed the `.hidden` class to show the modal, the ID selector's `display: none` remained active, keeping the modal hidden. Also removed the `#playerModal.flex` rule as `.bv-modal` already provides flex behavior. This fixes the issue where clicking a video card would trigger the logic but fail to display the player. --- css/tailwind.source.css | 5 ----- 1 file changed, 5 deletions(-) diff --git a/css/tailwind.source.css b/css/tailwind.source.css index 42327500..cb4372c0 100644 --- a/css/tailwind.source.css +++ b/css/tailwind.source.css @@ -3801,7 +3801,6 @@ position: fixed; inset: 0; z-index: var(--z-modal-player); - display: none; flex-direction: column; align-items: stretch; justify-content: flex-start; @@ -3822,10 +3821,6 @@ -webkit-overflow-scrolling: touch; } - #playerModal.flex { - display: block; - } - #modalVideo { width: 100%; aspect-ratio: 16 / 9;