@playpilot/tpi 8.29.2 → 8.29.3-beta.2

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.29.2",
+	"version": "8.29.3-beta.2",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

package/src/lib/injection.ts

@@ -355,6 +355,9 @@ function addLinkInjectionEventListeners(injections: LinkInjection[]): void {
 
 export function clearLinkInjections(): void {
   const elements = document.querySelectorAll(keySelector)
+
+  if (!elements.length) return
+
   elements.forEach((element) => clearLinkInjection(element.getAttribute(keyDataAttribute) || ''))
 
   clearAfterArticlePlaylinks()

package/src/lib/modal.ts

@@ -65,6 +65,8 @@ function addModalToList({ type = 'title', injection = null, data, scrollPosition
 
 /** Unmount the last modal is the list of modals but keep it in the list of active modals */
 export function closeCurrentModal(outro: boolean = true): void {
+  console.trace('closeCurrentModal')
+
   if (!modals.length) return
 
   saveCurrentModalScrollPosition()
@@ -80,6 +82,8 @@ export function destroyCurrentModal(outro: boolean = true): void {
 }
 
 export function destroyAllModals(outro: boolean = true): void {
+  console.trace('destroyAllModals')
+
   closeCurrentModal(outro)
 
   while (modals.length > 0) modals.pop()

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

@@ -8,7 +8,6 @@
   import { track } from '$lib/tracking'
   import type { ParticipantData } from '$lib/types/participant'
   import type { TitleData } from '$lib/types/title'
-  import { getContext } from 'svelte'
   import ParticipantImage from '../Participants/ParticipantImage.svelte'
   import Rail from './Rail.svelte'
   import { exploreModalUrl } from '$lib/routes'
@@ -20,8 +19,6 @@
 
   const { title, heading = t('Cast') }: Props = $props()
 
-  const navigate: Function | undefined = getContext('navigate')
-
   /**
    * Order participants by how often their name appears in an the page text. This only takes into account
    * their full name. If an actor is mentioned by first name only, which is often the case, they won't get

package/src/tests/lib/injection.test.js

@@ -573,7 +573,7 @@ describe('injection.ts', () => {
     })
 
     it('Should remove all popovers that may not have been removed properly in previous destroy attempts', async () => {
-      document.body.innerHTML = '<div data-playpilot-injection-popover></div> <p>This is a sentence with an injection.</p>'
+      document.body.innerHTML = '<div data-playpilot-injection-key></div><div data-playpilot-injection-popover></div> <p>This is a sentence with an injection.</p>'
 
       const elements = Array.from(document.body.querySelectorAll('p'))
       const injection = generateInjection('This is a sentence with an injection.', 'an injection')

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

@@ -93,7 +93,7 @@ describe('ParticipantsRail.svelte', () => {
     window.PlayPilotLinkInjections = {
       config: {
         open_tpi_links_in_explore: true,
-        explore_navigation_path: 'some-explore-url.com'
+        explore_navigation_path: 'some-explore-url.com',
       },
     }