refactor(html): remove temporary fullscreen test code from play button

Clean up development artifacts:
- Remove double-click event listener for fullscreen testing
- Remove temporary mediaStore reference and console logging
- Simplify handleEvent to only handle click for play/pause
- Restore play button to single responsibility (playback control only)

Play button now focuses solely on play/pause functionality while
fullscreen control is handled by dedicated fullscreen button.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Christian Pillsbury
2025-09-12 08:06:08 -07:00
committed by Christian Pillsbury
co-authored by Claude
parent 465342e431
commit 35072fd7c5
@@ -11,16 +11,18 @@ export class PlayButtonBase extends MediaChromeButton {
| { paused: boolean; requestPlay: () => void; requestPause: () => void }
| undefined;
constructor() {
super();
}
handleEvent(event: Event) {
const { type } = event;
const state = this._state;
if (state) {
if (type === 'click') {
if (state.paused) {
state.requestPlay();
} else {
state.requestPause();
}
if (state && type === 'click') {
if (state.paused) {
state.requestPlay();
} else {
state.requestPause();
}
}
}
@@ -29,7 +31,7 @@ export class PlayButtonBase extends MediaChromeButton {
return this._state?.paused;
}
_update(props: any, state: any) {
_update(props: any, state: any, _mediaStore?: any) {
this._state = state;
/** @TODO Follow up with React vs. W.C. data-* attributes discrepancies (CJP) */
// Make generic