chore(usevisibletask$): remove warnings

Author: maiieul

CODING_STANDARDS.md

@@ -50,14 +50,14 @@ export type TooltipProps = {
 };
 ```
 
-Note: if you want to include the HTML attributes, you can use the `QwikIntrinsicElements` type from `@builder.io/qwik` package and extend it with the [intersection type](https://www.typescriptlang.org/docs/handbook/2/objects.html#intersection-types).
+Note: if you want to include the HTML attributes, you can use the `PropsOf` type.
 
 Example:
 
 ```ts
-import { QwikIntrinsicElements } from '@builder.io/qwik';
+import { PropsOf } from '@builder.io/qwik';
 
-export type TooltipProps = QwikIntrinsicElements['div'] & {
+export type TooltipProps = PropsOf<'div'> & {
   tip: string;
   type?: ColorTypes;
   position?: Positions;

apps/website/src/components/highlight/highlight.tsx

@@ -27,6 +27,7 @@ export const Highlight = component$(
   }: HighlightProps) => {
     const codeSig = useSignal('');
 
+    // eslint-disable-next-line qwik/no-use-visible-task
     useVisibleTask$(
       async function createHighlightedCode() {
         // eslint-disable-next-line @typescript-eslint/no-explicit-any

apps/website/src/root.tsx

@@ -38,6 +38,7 @@ export default component$(() => {
 
   useContextProvider(APP_STATE_CONTEXT_ID, appState);
 
+  // eslint-disable-next-line qwik/no-use-visible-task
   useVisibleTask$(async () => {
     const userStoredTheme = localStorage.getItem(THEME_STORAGE_KEY);
     if (userStoredTheme) {

apps/website/src/routes/docs/headless/introduction/index.mdx

@@ -2,7 +2,7 @@
 title: Qwik UI | Introduction
 ---
 
-import headlessHero from '../../../../../public/images/qwik-ui-headless-hero.webp';
+import headlessHero from '/public/images/qwik-ui-headless-hero.webp';
 import { Note } from '~/components/note/note';
 import { FeatureList } from '~/components/feature-list/feature-list';
 import {

apps/website/src/routes/layout-landing.tsx

@@ -6,10 +6,10 @@ import { Footer } from '~/components/footer/footer';
 import { DocsNavigation } from '~/components/navigation-docs/navigation-docs';
 
 export default component$(() => {
-  // useStyles$(globalStyles);
-
   const state = useContext(OLD_APP_STATE_CONTEXT_ID);
   const rootStore = useAppState();
+
+  // eslint-disable-next-line qwik/no-use-visible-task
   useVisibleTask$(() => {
     state.darkMode = localStorage.getItem('theme') === 'dark';
   });

packages/kit-headless/src/components/accordion/accordion-content.tsx

@@ -70,6 +70,7 @@ export const AccordionContent = component$(({ ...props }: ContentProps) => {
   });
 
   /* calculates height of the content container based on children */
+  // eslint-disable-next-line qwik/no-use-visible-task
   useVisibleTask$(function calculateHeightVisibleTask({ track }) {
     if (animated === false) {
       return;

packages/kit-headless/src/components/accordion/accordion-root.tsx

@@ -6,7 +6,6 @@ import {
   useSignal,
   useTask$,
   useVisibleTask$,
-  type PropFunction,
   type QwikIntrinsicElements,
 } from '@builder.io/qwik';
 
@@ -18,8 +17,8 @@ export type AccordionRootProps = {
   animated?: boolean;
   enhance?: boolean;
   collapsible?: boolean;
-  onSelectedIndexChange$?: PropFunction<(index: number) => void>;
-  onFocusIndexChange$?: PropFunction<(index: number) => void>;
+  onSelectedIndexChange$?: (index: number) => void;
+  onFocusIndexChange$?: (index: number) => void;
 } & QwikIntrinsicElements['div'];
 
 export const AccordionRoot = component$(
@@ -103,6 +102,7 @@ export const AccordionRoot = component$(
     });
 
     // takes a role call of its children (reactive b/c it's a signal)
+    // eslint-disable-next-line qwik/no-use-visible-task
     useVisibleTask$(function reIndexTriggers() {
       updateTriggers$();
     });

packages/kit-headless/src/components/accordion/accordion-trigger.tsx

@@ -83,6 +83,7 @@ export const AccordionTrigger = component$(
       }
     });
 
+    // eslint-disable-next-line qwik/no-use-visible-task
     useVisibleTask$(function navigateTriggerVisibleTask({ cleanup }) {
       if (!triggerElement) {
         return;
@@ -105,6 +106,7 @@ export const AccordionTrigger = component$(
       });
     });
 
+    // eslint-disable-next-line qwik/no-use-visible-task
     useVisibleTask$(
       function cleanupTriggersTask({ cleanup }) {
         cleanup(() => {

packages/kit-headless/src/components/carousel/use/use-items.ts

@@ -115,6 +115,7 @@ export const useItems = (
     active.isLast.value = index + 1 === getCount(ref);
   });
 
+  // eslint-disable-next-line qwik/no-use-visible-task
   useVisibleTask$(() => {
     total.value = getCount(ref);
     scrollAt(active.current.value);

packages/kit-headless/src/components/carousel/use/use-pages.ts

@@ -113,6 +113,7 @@ export const usePages = (ref: Signal<HTMLElement | undefined>, params: Params):
     scrollAt(ranges.value.at(current.value + 1)?.[0] || 0);
   });
 
+  // eslint-disable-next-line qwik/no-use-visible-task
   useVisibleTask$(() => computePages());
   useOnWindow('resize', computePages);