Use bind:this for playerElement to avoid easy breaking

This commit is contained in:
WardPearce
2026-03-02 20:31:58 +13:00
parent 90417f798c
commit 6556e9aa1b
4 changed files with 4 additions and 7 deletions
@@ -82,7 +82,7 @@
<article
{...mergeAttrs(spatialItem.attrs, {
onclick: () => goToItem(uniqueItemId),
id: `item-${index}`
id: uniqueItemId
})}
class="no-padding item-select border"
class:item-select-focused={spatialItem.highlighted}
@@ -436,7 +436,6 @@
}
}
});
playerElement = document.getElementById('player') as HTMLMediaElement;
playerElement.loop = $playerAlwaysLoopStore;
@@ -839,7 +838,7 @@
<video
controls={false}
autoplay={$playerAutoPlayStore}
id="player"
bind:this={playerElement}
poster={getBestThumbnail(data.video.videoThumbnails, 9999, 9999)}
></video>
{#if isEmbed && !$isAndroidTvStore}
@@ -150,7 +150,7 @@
class:active={isActive(tab.id)}
class:surface-container-lowest={isActive(tab.id)}
class:surface-container-highest={!isActive(tab.id)}
{...mergeAttrs(tabCategories.getTrigger(tab.id), { id: tab.id })}
{...tabCategories.getTrigger(tab.id)}
>
<i>{tab.icon}</i>
<span>{tab.label}</span>
+1 -3
View File
@@ -382,9 +382,7 @@
{#if $navigating}
<PageLoading />
{:else}
<div tabindex="0">
{@render children?.()}
</div>
{@render children?.()}
{/if}
<SyncParty />