@poveste/app 0.13.0 → 0.14.0

package/src/app/util/preview-settings.ts

@@ -37,5 +37,5 @@ export function applyPreviewSettings(settings: PreviewSettings) {
 }
 
 export function getContrastColor(setting: PreviewSettings) {
-  return povesteConfig.backgroundPresets.find(preset => preset.color === setting.backgroundColor)?.contrastColor ?? 'unset'
+  return povesteConfig.backgroundPresets?.find(preset => preset.color === setting.backgroundColor)?.contrastColor ?? 'unset'
 }

package/src/app/util/scroll.ts

@@ -2,7 +2,11 @@ import type { Ref } from 'vue'
 import scrollIntoView from 'scroll-into-view-if-needed'
 import { onMounted, watch } from 'vue'
 
-export function useScrollOnActive(active: Ref<boolean>, el: Ref<HTMLElement>) {
+// `el` is a template ref, so it is undefined until the component mounts and
+// again once it unmounts. `autoScroll` has always guarded for that; the
+// signature was the only thing claiming otherwise, and it made four callers
+// pass a `Ref<HTMLDivElement | undefined>` into a parameter that denied it.
+export function useScrollOnActive(active: Ref<boolean>, el: Ref<HTMLElement | undefined>) {
   watch(active, (value) => {
     if (value) {
       autoScroll()

package/src/app/util/state.ts

@@ -19,7 +19,7 @@ export function toRawDeep(val, clean = false, seen = new WeakMap()) {
   }
 
   if (Array.isArray(unwrappedValue)) {
-    const result = []
+    const result: unknown[] = []
     seen.set(unwrappedValue, result)
     let list = unwrappedValue.map(value => toRawDeep(value, clean, seen))
     if (clean) {

package/src/app/util/variant.ts

@@ -11,7 +11,10 @@ import { useRoute } from 'vue-router'
  * reader to pick one" needs the same answer it will give — hence one predicate
  * rather than two copies of the condition (#328).
  */
-export function autoSelectsVariant(story: Story | null | undefined): boolean {
+// A type predicate rather than `boolean`: the body already establishes that the
+// story is there, and returning `boolean` threw that away at every call site —
+// `StoryView` reads the story twice immediately after this returns true.
+export function autoSelectsVariant(story: Story | null | undefined): story is Story {
   return !!story && (!!story.lastSelectedVariant || story.variants.length === 1)
 }
 

package/src/app/components/base/BaseCopyIcon.vue

@@ -1,34 +0,0 @@
-<script lang="ts">
-export default {
-  name: 'HstCopyIcon',
-}
-</script>
-
-<script lang="ts" setup>
-import { Icon } from '@iconify/vue'
-import { useClipboard } from '@vueuse/core'
-import { VTooltip as vTooltip } from 'floating-vue'
-
-const props = defineProps<{
-  content: string
-}>()
-
-const { copy, copied } = useClipboard()
-
-const action = () => copy(props.content)
-</script>
-
-<template>
-  <Icon
-    v-tooltip="{
-      content: 'Copied!',
-      triggers: [],
-      shown: copied,
-      distance: 12,
-      delay: 0,
-    }"
-    icon="carbon:copy-file"
-    class="poveste-base-copy-icon w-4 h-4 opacity-50 hover:opacity-100 hover:text-primary-500 cursor-pointer"
-    @click="action()"
-  />
-</template>