Fix stale spacer-observe comment: drop visualizer/clipping ref, name spacer as sole consumer

This commit is contained in:
daniel-c-harvey
2026-06-24 10:40:52 -04:00
parent cb899a2913
commit adbd376d42
@@ -246,13 +246,15 @@ public partial class AudioPlayerBar : ComponentBase, IAsyncDisposable
}
// For the docked player: we observe in BOTH expanded and minimized states
// so --player-height always reflects the live height of whichever element
// is visible. This keeps the WaveformVisualizer clipped to the top of
// the footer in both states (fix §1).
// so --player-height stays non-zero and always reflects the live height of
// whichever element is visible. The var's sole live consumer is the
// player-spacer's .expanded height (keeps the spacer sized correctly across
// breakpoints and banner reflows).
// expanded → observe _playerRoot (full player bar, reflows across breakpoints)
// minimized → observe _miniDock (floating FAB container, ~5660px)
// The player-spacer's .minimized class uses a hardcoded height and ignores
// the var, so publishing the FAB height here does not regress the spacer.
// The player-spacer's .minimized class uses a hardcoded 60px and ignores
// the var, so observing in minimized state is belt-and-braces; it does not
// regress the spacer.
var elementToObserve = _isMinimized ? _miniDock : _playerRoot;
var alreadyOnThisElement = _spacerObserved && elementToObserve.Id == _lastObservedElement.Id;