drop-shadow-sm (previous commit) didn't fix the corner artifact in
testing, so revert to the simplest option instead of chasing which
rasterization path is actually at fault: video tiles just don't get
an offset shadow. Border stays.
Reverts the --drop-shadow-* token additions in base.css/clay.css/
navy.css from the previous commit since nothing else used them.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
With exactly 2 video tiles, the layout branch flex-shrinks tiles to
exactly fill the scroll container (fractional heights, bottom tile
flush against the scroller's clip edge). In that configuration
Chromium's rounded-rect box-shadow fast path misrenders at fractional
pixel offsets: the corner shows a straight diagonal wedge instead of
a rounded crescent, and the bottom tile's bottom shadow band is
clipped entirely by the scroller's overflow. With 3+ tiles the grid
branch uses natural (non-fractional) 16:9 heights, so it doesn't hit
this.
filter: drop-shadow() renders the same offset shadow through a
different rasterization path that isn't susceptible to this, and
(as a side benefit) isn't clipped by scrollable overflow the way
box-shadow's ink overflow is.
- VideoCallContent.svelte: shadow-sm -> drop-shadow-sm on video tiles.
- base.css: add --drop-shadow/-sm/-lg tokens mirroring the --shadow-*
scale, consumed by Tailwind's drop-shadow-* utilities.
- clay.css, navy.css: matching --drop-shadow-* overrides in both
light and dark. flat.css needs no changes; it inherits base.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
9a80d059 (Migrate to new welshman) accidentally committed
link:../welshman/packages/* pnpm overrides into pnpm-workspace.yaml,
forcing resolution to a local welshman checkout that doesn't exist in
this environment. package.json already specifies the correct upstream
range (^0.9.0-pre2), so a fresh `pnpm install` produced dangling
node_modules/@welshman/* symlinks and broke `pnpm run dev`.
Drop the overrides (keep sharp) and regenerate pnpm-lock.yaml, which
now resolves upstream @welshman/*@0.9.0-pre5.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- Remove the tile.track ternary from the name label class — use bg-backdrop/80
consistently for all tiles (both video and no-video)
- Remove the !tile.track conditional from the border — apply border border-line
to all tiles consistently
- Fixes shadow rendering inconsistency: with two tiles, the conditional border
+ ternary caused asymmetric shadow rendering (cropped/pointed shadows)
Per review feedback from @matt on PR #3 (forgejo).
- Add border (border-line) to no-video tile outer div for card contrast
against bg-surface panel (video-on tiles unchanged)
- Use bg-backdrop/80 for name label on no-video tiles for legibility
(video-on tiles keep bg-surface/80)
Part of flotilla-j4k / coracle/flotilla#343
