@playpilot/tpi 8.34.4 → 8.35.0-beta.1

package/src/lib/enums/TrackingEvent.ts

@@ -68,6 +68,7 @@ export const TrackingEvent = {
   RegionRequestFailed: 'ali_region_request_failed',
   YouTubeAvailabilityRequestFailed: 'ali_youtube_availability_request_failed',
   CardToExploreCta: 'ali_card_to_explore_cta',
+  TitleParticipantsView: 'ali_title_participants_view',
 
   // Explore
   // These deliberately do not use the `ali_` prefix. We might want to separate this from TPI at some point

package/src/routes/components/Explore/Filter/Filter.svelte

@@ -1,148 +1,148 @@
-<script lang="ts">
-  import type { ExploreFilter } from '$lib/types/filter'
-  import genres from '$lib/data/genres.json'
-  import { fetchProviders } from '$lib/api/providers'
-  import { t } from '$lib/localization'
-  import FilterItem from './FilterItem.svelte'
-  import FilterSorting from './FilterSorting.svelte'
-  import Button from '../../Button.svelte'
-  import IconArrow from '../../Icons/IconArrow.svelte'
-  import IconFilter from '../../Icons/IconFilter.svelte'
-  import { fly, scale } from 'svelte/transition'
-  import ActiveFilterItems from './ActiveFilterItems.svelte'
-  import { isFilterItemActive } from '$lib/filter'
-  import { cdnBaseUrl } from '$lib/constants'
-
-  interface Props {
-    filter: ExploreFilter
-    limit?: boolean
-    showSorting?: boolean
-    searchQuery?: string
-    onchange?: () => void
-    onempty?: () => void
-  }
-
-  const {
-    filter,
-    limit = true,
-    showSorting = true,
-    searchQuery = '',
-    onchange = () => null,
-    onempty = () => null,
-  }: Props = $props()
-
-  const shownItemsLimit = 3
-  const items = [{
-    label: t('Services'),
-    param: 'providers',
-    fetchData: async () => (await fetchProviders()).map(playlink => ({ label: playlink.name, value: playlink.slug })),
-  }, {
-    label: t('Genres'),
-    param: 'genres',
-    data: genres.filter(genre => parseInt(genre.slug) < 900).map(genre => ({ label: t(genre.name), value: genre.slug })),
-  }, {
-    label: t('IMDb'),
-    param: 'imdb_score',
-    range: [0, 10] as [number, number],
-  }, {
-    label: t('Year'),
-    param: 'year',
-    range: [1900, new Date().getFullYear()] as [number, number],
-  }, {
-    label: t('Runtime'),
-    param: 'length',
-    range: [0, 180] as [number, number],
-    valueAppend: t('Minutes'),
-  }, {
-    label: t('Countries'),
-    param: 'country_codes',
-    fetchData: async () => import.meta.env.DEV ?
-      (await import('$lib/data/countries.json')).default :
-      (await fetch(`${cdnBaseUrl}/src/lib/data/countries.json`)).json(),
-  }, {
-    label: t('Category'),
-    param: 'category',
-    data: [{ label: t('Movies'), value: 'movie' }, { label: t('Shows'), value: 'series' }, { label: t('Kids'), value: 'kids' }, { label: t('Documentary'), value: 'documentary' }],
-  }]
-
-  let limited = $state(limit)
-  let wasPreviouslyActive = false
-
-  $effect(() => {
-    const hasAnyActiveFilter = Object.keys(filter).some((param) => {
-      const { range } = items.find((i: any) => param === i.param) || {}
-
-      return isFilterItemActive(filter[param], range)
-    })
-
-    if (hasAnyActiveFilter || searchQuery) {
-      wasPreviouslyActive = true
-    } else if (wasPreviouslyActive) {
-      wasPreviouslyActive = false
-      onempty()
-    }
-  })
-</script>
-
-<div class="filter" role="navigation" class:limit>
-  <div class="items">
-    {#each limited ? items.slice(0, shownItemsLimit) : items as item}
-      <div transition:scale={{ start: 0.75, duration: 100 }}>
-        <FilterItem {filter} {onchange} {...item} />
-      </div>
-    {/each}
-  </div>
-
-  <div class="actions">
-    {#if limit}
-      <Button variant="link" onclick={() => limited = !limited}>
-        <IconFilter />
-        {t('All Filters')}
-        <IconArrow direction={limited ? 'down' : 'up'} />
-      </Button>
-    {/if}
-
-    {#if showSorting}
-      <div class="sorting" transition:fly={{ x: 5, duration: 100 }}>
-        <FilterSorting {filter} {onchange} />
-      </div>
-    {/if}
-  </div>
-</div>
-
-<ActiveFilterItems {filter} {items} />
-
-<style lang="scss">
-  .filter {
-    position: relative;
-  }
-
-  .items {
-    display: flex;
-    flex-wrap: wrap;
-    gap: margin(0.5);
-    padding-right: margin(7);
-
-    .limit & {
-      padding-right: 0;
-    }
-  }
-
-  .actions {
-    position: absolute;
-    top: margin(0.25);
-    right: 0;
-
-    .limit & {
-      position: initial;
-      top: auto;
-      display: flex;
-      justify-content: space-between;
-      margin-top: margin(0.5);
-    }
-  }
-
-  .sorting {
-    margin-top: margin(0.1);
-  }
-</style>
+<script lang="ts">
+  import type { ExploreFilter } from '$lib/types/filter'
+  import genres from '$lib/data/genres.json'
+  import { fetchProviders } from '$lib/api/providers'
+  import { t } from '$lib/localization'
+  import FilterItem from './FilterItem.svelte'
+  import FilterSorting from './FilterSorting.svelte'
+  import Button from '../../Button.svelte'
+  import IconArrow from '../../Icons/IconArrow.svelte'
+  import IconFilter from '../../Icons/IconFilter.svelte'
+  import { fly, scale } from 'svelte/transition'
+  import ActiveFilterItems from './ActiveFilterItems.svelte'
+  import { isFilterItemActive } from '$lib/filter'
+  import { cdnBaseUrl } from '$lib/constants'
+
+  interface Props {
+    filter: ExploreFilter
+    limit?: boolean
+    showSorting?: boolean
+    searchQuery?: string
+    onchange?: () => void
+    onempty?: () => void
+  }
+
+  const {
+    filter,
+    limit = true,
+    showSorting = true,
+    searchQuery = '',
+    onchange = () => null,
+    onempty = () => null,
+  }: Props = $props()
+
+  const shownItemsLimit = 3
+  const items = [{
+    label: t('Services'),
+    param: 'providers',
+    fetchData: async () => (await fetchProviders()).map(playlink => ({ label: playlink.name, value: playlink.slug })),
+  }, {
+    label: t('Genres'),
+    param: 'genres',
+    data: genres.filter(genre => parseInt(genre.slug) < 900).map(genre => ({ label: t(genre.name), value: genre.slug })),
+  }, {
+    label: t('IMDb'),
+    param: 'imdb_score',
+    range: [0, 10] as [number, number],
+  }, {
+    label: t('Year'),
+    param: 'year',
+    range: [1900, new Date().getFullYear()] as [number, number],
+  }, {
+    label: t('Runtime'),
+    param: 'length',
+    range: [0, 180] as [number, number],
+    valueAppend: t('Minutes'),
+  }, {
+    label: t('Countries'),
+    param: 'country_codes',
+    fetchData: async () => import.meta.env.DEV ?
+      (await import('$lib/data/countries.json')).default :
+      (await fetch(`${cdnBaseUrl}/src/lib/data/countries.json`)).json(),
+  }, {
+    label: t('Category'),
+    param: 'category',
+    data: [{ label: t('Movies'), value: 'movie' }, { label: t('Shows'), value: 'series' }, { label: t('Kids'), value: 'kids' }, { label: t('Documentary'), value: 'documentary' }],
+  }]
+
+  let limited = $state(limit)
+  let wasPreviouslyActive = false
+
+  $effect(() => {
+    const hasAnyActiveFilter = Object.keys(filter).some((param) => {
+      const { range } = items.find((i: any) => param === i.param) || {}
+
+      return isFilterItemActive(filter[param], range)
+    })
+
+    if (hasAnyActiveFilter || searchQuery) {
+      wasPreviouslyActive = true
+    } else if (wasPreviouslyActive) {
+      wasPreviouslyActive = false
+      onempty()
+    }
+  })
+</script>
+
+<div class="filter" role="navigation" class:limit>
+  <div class="items">
+    {#each limited ? items.slice(0, shownItemsLimit) : items as item}
+      <div transition:scale={{ start: 0.75, duration: 100 }}>
+        <FilterItem {filter} {onchange} {...item} />
+      </div>
+    {/each}
+  </div>
+
+  <div class="actions">
+    {#if limit}
+      <Button variant="link" onclick={() => limited = !limited}>
+        <IconFilter />
+        {t('All Filters')}
+        <IconArrow direction={limited ? 'down' : 'up'} />
+      </Button>
+    {/if}
+
+    {#if showSorting}
+      <div class="sorting" transition:fly={{ x: 5, duration: 100 }}>
+        <FilterSorting {filter} {onchange} />
+      </div>
+    {/if}
+  </div>
+</div>
+
+<ActiveFilterItems {filter} {items} />
+
+<style lang="scss">
+  .filter {
+    position: relative;
+  }
+
+  .items {
+    display: flex;
+    flex-wrap: wrap;
+    gap: margin(0.5);
+    padding-right: margin(7);
+
+    .limit & {
+      padding-right: 0;
+    }
+  }
+
+  .actions {
+    position: absolute;
+    top: margin(0.25);
+    right: 0;
+
+    .limit & {
+      position: initial;
+      top: auto;
+      display: flex;
+      justify-content: space-between;
+      margin-top: margin(0.5);
+    }
+  }
+
+  .sorting {
+    margin-top: margin(0.1);
+  }
+</style>

package/src/routes/components/Explore/Routes/ExploreResults.svelte

@@ -20,9 +20,10 @@
   interface Props {
     searchQuery?: string,
     filter?: ExploreFilter
+    forceGrid?: boolean
   }
 
-  const { searchQuery = '', filter = {} }: Props = $props()
+  const { searchQuery = '', filter = {}, forceGrid = false }: Props = $props()
 
   // svelte-ignore non_reactive_update
   let page = 1
@@ -33,12 +34,13 @@
   let promise = $state(getTitlesForFilter())
   let width = $state(0)
 
-  const grid = $derived(width > 500)
+  const grid = $derived(forceGrid || width > 500)
   const gridColumns = $derived.by(() => {
     if (width > 1400) return 8
     if (width > 900) return 6
-    else if (width > 600) return 4
-    return 3
+    if (width > 600) return 4
+    if (width > 400) return 3
+    return 2
   })
   const TitleComponent = $derived(grid ? GridTitle : ListTitle)
   const SkeletonComponent = $derived(grid ? GridTitleSkeleton : ListTitleSkeleton)

package/src/routes/components/Explore/Routes/ExploreTitle.svelte

@@ -28,7 +28,7 @@
 
 {#snippet back(offset = false)}
   <div class="back" class:offset>
-    <Button variant="link" onclick={() => { navigate('home') }}>
+    <Button variant="link" onclick={() => navigate('home')}>
       <IconArrow direction="left" />
       {t('Home')}
     </Button>

package/src/routes/components/Modals/Modal.svelte

@@ -215,6 +215,10 @@
     &.background-blur {
       backdrop-filter: blur(1.5rem);
     }
+
+    :global(*) {
+      box-sizing: border-box;
+    }
   }
 
   .dialog,

package/src/routes/components/Rails/ParticipantsRail.svelte

@@ -52,9 +52,17 @@
 
     window.location.href = exploreModalUrl(participant.sid)
   }
+
+  async function getParticipants(): Promise<ParticipantData[]> {
+    const participants = await fetchParticipantsForTitle(title)
+
+    if (participants?.length) track(TrackingEvent.TitleParticipantsView, title, { participants: participants.map(p => p.name).slice(0, 20) })
+
+    return participants
+  }
 </script>
 
-{#await fetchParticipantsForTitle(title)}
+{#await getParticipants()}
   <Rail {heading}>
     {#each { length: 5 }}
       <div class="participant">

package/src/tests/routes/components/Rails/ParticipantsRail.test.js

@@ -119,4 +119,14 @@ describe('ParticipantsRail.svelte', () => {
 
     expect(track).toHaveBeenCalledWith(TrackingEvent.ParticipantClick, null, { title_source: title.original_title, participant: participants[0].name })
   })
+
+  it('Should fire track event after participants are fetched', async () => {
+    vi.mocked(fetchParticipantsForTitle).mockResolvedValueOnce(participants)
+
+    render(ParticipantsRail, { title })
+
+    await waitFor(async () => {
+      expect(track).toHaveBeenCalledWith(TrackingEvent.TitleParticipantsView, title, { participants: participants.map(p => p.name) })
+    })
+  })
 })