fix(calendar): improve onboarding UX — clearer progress, better selection, apply preferences

- Replace thin progress bar with animated step dots
- Add checkmark icon and stronger highlight for selected options
- Remove redundant step emoji (duplicate of header)
- Hide "Zurück" button on first step instead of just opacity:0
- Remove timezone step (auto-detect is always correct, 4 options too limited)
- Replace unsupported "day" view with "agenda" in view options
- Wire onComplete to actually apply weekStart and defaultView to settingsStore

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Till JS 2026-03-21 11:00:28 +01:00
parent 8a0ab38958
commit 4770e15499
2 changed files with 67 additions and 76 deletions

View file

@ -1,5 +1,7 @@
import { createAppOnboardingStore, type AppOnboardingStep } from '@manacore/shared-app-onboarding';
import { userSettings } from './user-settings.svelte';
import { settingsStore } from './settings.svelte';
import type { CalendarViewType } from '@calendar/shared';
/**
* Calendar-specific onboarding steps
@ -36,12 +38,6 @@ const calendarOnboardingSteps: AppOnboardingStep[] = [
emoji: '👁️',
gradient: { from: 'indigo-500', to: 'indigo-700' },
options: [
{
id: 'day',
label: 'Tagesansicht',
description: 'Detaillierte 24-Stunden-Timeline',
emoji: '📆',
},
{
id: 'week',
label: 'Wochenansicht',
@ -54,44 +50,15 @@ const calendarOnboardingSteps: AppOnboardingStep[] = [
description: 'Kompakte Monatsübersicht',
emoji: '📅',
},
{
id: 'agenda',
label: 'Agenda',
description: 'Chronologische Terminliste',
emoji: '📋',
},
],
defaultValue: 'week',
},
{
id: 'timezone',
type: 'select',
question: 'Welche Zeitzone verwendest du?',
description: 'Termine werden in dieser Zeitzone angezeigt.',
emoji: '🌍',
gradient: { from: 'emerald-500', to: 'emerald-700' },
options: [
{
id: 'auto',
label: 'Automatisch erkennen',
description: 'Basierend auf deinem Standort',
emoji: '📍',
},
{
id: 'Europe/Berlin',
label: 'Berlin (MEZ/MESZ)',
description: 'Deutschland, Österreich, Schweiz',
emoji: '🇩🇪',
},
{
id: 'Europe/London',
label: 'London (GMT/BST)',
description: 'Großbritannien',
emoji: '🇬🇧',
},
{
id: 'America/New_York',
label: 'New York (EST/EDT)',
description: 'US-Ostküste',
emoji: '🇺🇸',
},
],
defaultValue: 'auto',
},
{
id: 'welcome',
type: 'info',
@ -132,13 +99,25 @@ export const calendarOnboarding = createAppOnboardingStore({
steps: calendarOnboardingSteps,
userSettings,
onComplete: async (preferences) => {
console.log('[Calendar] Onboarding completed with preferences:', preferences);
// Apply week start preference
if (preferences.weekStart === 'monday') {
settingsStore.set('weekStartsOn', 1);
} else if (preferences.weekStart === 'sunday') {
settingsStore.set('weekStartsOn', 0);
}
// Apply preferences to the app
// The preferences are automatically saved to deviceSettings by the store
// Additional app-specific logic can go here (e.g., applying timezone, view settings)
// Apply default view preference
const viewMap: Record<string, CalendarViewType> = {
week: 'week',
month: 'month',
agenda: 'agenda',
};
const selectedView = preferences.defaultView as string;
if (selectedView && viewMap[selectedView]) {
settingsStore.set('defaultView', viewMap[selectedView]);
}
},
onSkip: async () => {
console.log('[Calendar] Onboarding skipped');
// Defaults are already sensible, nothing to do
},
});

View file

@ -1,5 +1,9 @@
<script lang="ts">
import type { AppOnboardingStore, AppOnboardingSelectStep, AppOnboardingToggleStep } from './types';
import type {
AppOnboardingStore,
AppOnboardingSelectStep,
AppOnboardingToggleStep,
} from './types';
interface Props {
store: AppOnboardingStore;
@ -65,12 +69,15 @@
</button>
</div>
<!-- Progress bar -->
<div class="h-1 bg-muted rounded-full overflow-hidden">
<div
class="h-full bg-primary transition-all duration-300 ease-out rounded-full"
style="width: {store.progress}%"
></div>
<!-- Step dots -->
<div class="flex items-center gap-1.5">
{#each Array(store.totalSteps) as _, i}
<div
class="h-1.5 rounded-full transition-all duration-300 {i <= store.currentStep
? 'bg-primary'
: 'bg-muted'} {i === store.currentStep ? 'w-6' : 'w-1.5'}"
></div>
{/each}
</div>
</header>
@ -80,18 +87,6 @@
{@const step = store.currentStepConfig}
<div class="text-center">
<!-- Step icon -->
{#if step.emoji}
{@const gradient = step.gradient || { from: 'primary', to: 'primary/70' }}
<div class="mb-5">
<div
class="inline-flex h-14 w-14 rounded-xl bg-gradient-to-br from-{gradient.from} to-{gradient.to} items-center justify-center shadow-lg"
>
<span class="text-2xl">{step.emoji}</span>
</div>
</div>
{/if}
<!-- Question -->
<h2 class="text-lg font-bold mb-2">{step.question}</h2>
{#if step.description}
@ -104,22 +99,34 @@
{@const selectStep = step as AppOnboardingSelectStep}
<div class="space-y-2">
{#each selectStep.options as option}
{@const isSelected = store.preferences[step.id] === option.id}
<button
onclick={() => selectOption(step.id, option.id)}
class="w-full p-3 rounded-xl border-2 transition-all text-left {store.preferences[step.id] === option.id
? 'border-primary bg-primary/10'
class="w-full p-3 rounded-xl border-2 transition-all text-left {isSelected
? 'border-primary bg-primary/15 shadow-sm shadow-primary/20'
: 'border-border hover:border-primary/50 bg-surface-elevated-1'}"
>
<div class="flex items-center gap-3">
{#if option.emoji}
<span class="text-xl">{option.emoji}</span>
{/if}
<div>
<div class="flex-1">
<p class="font-medium text-sm">{option.label}</p>
{#if option.description}
<p class="text-xs text-muted-foreground">{option.description}</p>
{/if}
</div>
{#if isSelected}
<svg
class="w-5 h-5 text-primary flex-shrink-0"
fill="none"
viewBox="0 0 24 24"
stroke="currentColor"
stroke-width="2.5"
>
<path stroke-linecap="round" stroke-linejoin="round" d="M5 13l4 4L19 7" />
</svg>
{/if}
</div>
</button>
{/each}
@ -135,7 +142,9 @@
>
<div class="flex items-center justify-between">
<span class="font-medium text-sm">
{isEnabled ? (toggleStep.enabledLabel || 'Aktiviert') : (toggleStep.disabledLabel || 'Deaktiviert')}
{isEnabled
? toggleStep.enabledLabel || 'Aktiviert'
: toggleStep.disabledLabel || 'Deaktiviert'}
</span>
<div
class="relative w-11 h-6 rounded-full transition-colors {isEnabled
@ -170,13 +179,16 @@
<!-- Footer with navigation -->
<footer class="border-t px-5 py-3 flex-shrink-0">
<div class="flex justify-between">
<button
onclick={handlePrev}
disabled={store.isFirstStep}
class="px-4 py-2 text-sm border rounded-lg hover:bg-muted transition-colors disabled:opacity-0 disabled:pointer-events-none"
>
Zurück
</button>
{#if store.isFirstStep}
<div></div>
{:else}
<button
onclick={handlePrev}
class="px-4 py-2 text-sm border rounded-lg hover:bg-muted transition-colors"
>
Zurück
</button>
{/if}
<button
onclick={handleNext}
disabled={store.saving}