fix: improve usability and ensure type safety (#97)

* fix(appearance): Remove duplicate initializeTheme call

initializeTheme is already called in app.ts

* fix(types): Type string is not assignable to type Method | undefined

Replaced the generic string type with the Method type from @inertiajs/core for better type safety and alignment with Inertia.js.

* fix(sidebar): Fix breadcrumbs rendering

Ensure the sidebar only renders breadcrumbs when the `breadcrumbs` object exists and has entries. This prevents potential runtime errors if `breadcrumbs` is null or undefined.

Author: firebed

resources/js/components/AppSidebarHeader.vue

@@ -16,7 +16,7 @@ withDefaults(defineProps<{
     >
         <div class="flex items-center gap-2">
             <SidebarTrigger class="-ml-1" />
-            <template v-if="breadcrumbs.length > 0">
+            <template v-if="breadcrumbs && breadcrumbs.length > 0">
                 <Breadcrumbs :breadcrumbs="breadcrumbs" />
             </template>
         </div>

resources/js/components/TextLink.vue

@@ -1,10 +1,11 @@
 <script setup lang="ts">
+import { Method } from '@inertiajs/core';
 import { Link } from '@inertiajs/vue3';
 
 interface Props {
     href: string;
     tabindex?: number;
-    method?: string;
+    method?: Method;
     as?: string;
 }
 

resources/js/composables/useAppearance.ts

@@ -66,8 +66,6 @@ export function useAppearance() {
     const appearance = ref<Appearance>('system');
 
     onMounted(() => {
-        initializeTheme();
-
         const savedAppearance = localStorage.getItem('appearance') as Appearance | null;
 
         if (savedAppearance) {