Minor viewer renames/fixes.

This commit is contained in:
Dermot Duffy
2023-01-24 19:36:54 -08:00
parent 208795d7b3
commit 3e749d03cb
+10 -14
View File
@@ -1,5 +1,5 @@
import { Task } from '@lit-labs/task'; import { Task } from '@lit-labs/task';
import { EmblaOptionsType, EmblaPluginType } from 'embla-carousel'; import { EmblaPluginType } from 'embla-carousel';
import { WheelGesturesPlugin } from 'embla-carousel-wheel-gestures'; import { WheelGesturesPlugin } from 'embla-carousel-wheel-gestures';
import { import {
CSSResultGroup, CSSResultGroup,
@@ -243,7 +243,7 @@ export class FrigateCardViewerCarousel extends LitElement {
} }
const promises: Promise<ResolvedMedia | null>[] = []; const promises: Promise<ResolvedMedia | null>[] = [];
view.queryResults?.getResults()?.forEach((media: ViewMedia) => { view.queryResults?.getResults()?.forEach((media: ViewMedia) => {
const mediaContentID = media.getContentID(cameras.get(media.getCameraID())); const mediaContentID = media.getContentID();
if (this.hass && mediaContentID) { if (this.hass && mediaContentID) {
promises.push( promises.push(
resolveMedia(this.hass, mediaContentID, this.resolvedMediaCache), resolveMedia(this.hass, mediaContentID, this.resolvedMediaCache),
@@ -266,7 +266,7 @@ export class FrigateCardViewerCarousel extends LitElement {
// on media load, since the media may or may not have been loaded at // on media load, since the media may or may not have been loaded at
// this point). // this point).
if (this.view?.context?.mediaViewer !== oldView?.context?.mediaViewer) { if (this.view?.context?.mediaViewer !== oldView?.context?.mediaViewer) {
this._recordingSeekHandler(); this._seekHandler();
} }
} }
super.updated(changedProperties); super.updated(changedProperties);
@@ -458,9 +458,7 @@ export class FrigateCardViewerCarousel extends LitElement {
} }
const media = this.view.queryResults?.getResult(index); const media = this.view.queryResults?.getResult(index);
const mediaContentID = media const mediaContentID = media ? media.getContentID() : null;
? media.getContentID(this.cameras.get(media.getCameraID()))
: null;
if (!mediaContentID) { if (!mediaContentID) {
return; return;
} }
@@ -518,7 +516,7 @@ export class FrigateCardViewerCarousel extends LitElement {
return false; return false;
} }
for (const media of this.view?.queryResults?.getResults() ?? []) { for (const media of this.view?.queryResults?.getResults() ?? []) {
const mediaContentID = media.getContentID(this.cameras.get(media.getCameraID())); const mediaContentID = media.getContentID();
if (mediaContentID && !this.resolvedMediaCache.has(mediaContentID)) { if (mediaContentID && !this.resolvedMediaCache.has(mediaContentID)) {
return false; return false;
} }
@@ -593,15 +591,14 @@ export class FrigateCardViewerCarousel extends LitElement {
.selected=${this.view?.queryResults?.getSelectedIndex() ?? 0} .selected=${this.view?.queryResults?.getSelectedIndex() ?? 0}
transitionEffect=${this._getTransitionEffect()} transitionEffect=${this._getTransitionEffect()}
@frigate-card:media-carousel:select=${this._setViewHandler.bind(this)} @frigate-card:media-carousel:select=${this._setViewHandler.bind(this)}
@frigate-card:media:loaded=${this._recordingSeekHandler.bind(this)} @frigate-card:media:loaded=${this._seekHandler.bind(this)}
> >
<frigate-card-next-previous-control <frigate-card-next-previous-control
slot="previous" slot="previous"
.hass=${this.hass} .hass=${this.hass}
.direction=${'previous'} .direction=${'previous'}
.controlConfig=${this.viewerConfig?.controls.next_previous} .controlConfig=${this.viewerConfig?.controls.next_previous}
.thumbnail=${prev?.getThumbnail(this.cameras.get(prev.getCameraID())) ?? .thumbnail=${prev?.getThumbnail() ?? undefined}
undefined}
.label=${prev?.getTitle() ?? ''} .label=${prev?.getTitle() ?? ''}
?disabled=${!prev} ?disabled=${!prev}
@click=${(ev) => { @click=${(ev) => {
@@ -615,8 +612,7 @@ export class FrigateCardViewerCarousel extends LitElement {
.hass=${this.hass} .hass=${this.hass}
.direction=${'next'} .direction=${'next'}
.controlConfig=${this.viewerConfig?.controls.next_previous} .controlConfig=${this.viewerConfig?.controls.next_previous}
.thumbnail=${next?.getThumbnail(this.cameras.get(next.getCameraID())) ?? .thumbnail=${next?.getThumbnail() ?? undefined}
undefined}
.label=${next?.getTitle() ?? ''} .label=${next?.getTitle() ?? ''}
?disabled=${!next} ?disabled=${!next}
@click=${(ev) => { @click=${(ev) => {
@@ -630,7 +626,7 @@ export class FrigateCardViewerCarousel extends LitElement {
/** /**
* Fire a media show event when a slide is selected. * Fire a media show event when a slide is selected.
*/ */
protected _recordingSeekHandler(): void { protected _seekHandler(): void {
const selectedIndex = this.view?.queryResults?.getSelectedIndex() ?? null; const selectedIndex = this.view?.queryResults?.getSelectedIndex() ?? null;
const seek = const seek =
selectedIndex !== null selectedIndex !== null
@@ -655,7 +651,7 @@ export class FrigateCardViewerCarousel extends LitElement {
} }
const lazyLoad = this.viewerConfig.lazy_load; const lazyLoad = this.viewerConfig.lazy_load;
const mediaContentID = media.getContentID(this.cameras.get(media.getCameraID())); const mediaContentID = media.getContentID();
const resolvedMedia = mediaContentID const resolvedMedia = mediaContentID
? this.resolvedMediaCache?.get(mediaContentID) ? this.resolvedMediaCache?.get(mediaContentID)
: null; : null;