ReviewOS

also looking at this

loghqorg/loghq

chore(release): promote UI, accessibility, and STX updates to production

#11
Merged glennmichael123 wants to merge main into production
16 files +176 -67

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

resources/views/analytics.stxmodified+13-7
Changes to resources/views/analytics.stx
@@ -382,10 +382,13 @@ summary:focus-visible {
382382 client pre-paint bounce (deleted with AuthGuard) the server renders a
383383 sign-in panel when the auth-token cookie resolved no user. --}}
384384 @if (!user)
385 <div class="py-24 text-center">
386 <p class="mb-4 text-sm text-subtle">You're signed out.</p>
385 <EmptyState
386 variant="bare"
387 title="Sign in required"
388 description="You need to be signed in to see your analytics."
389 >
387390 <StxLink to="/login" class="inline-block px-4 py-2.5 font-semibold text-accent-ink text-sm bg-accent hover:bg-[color-mix(in_srgb,var(--accent)_88%,#000)] rounded-[10px]">Sign in</StxLink>
388 </div>
391 </EmptyState>
389392 @endif
390393
391394 @if (user && projectsUnavailable)
@@ -396,10 +399,13 @@ summary:focus-visible {
396399 @endif
397400
398401 @if (user && !projectsUnavailable && !projectId)
399 <div class="py-24 text-center">
400 <p class="text-muted text-sm">No projects yet.</p>
401 <p class="mt-2 text-subtle text-xs"><StxLink to="/projects/new" class="text-accent">Create one</StxLink> and its analytics appear here.</p>
402 </div>
402 <EmptyState
403 variant="bare"
404 title="No projects yet"
405 description="Create one and its analytics appear here."
406 >
407 <StxLink to="/projects/new" class="text-accent">Create an app</StxLink>
408 </EmptyState>
403409 @endif
404410
405411 @if (user && projectId)