also looking at this
chore(release): promote UI, accessibility, and STX updates to production
#11
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.
| @@ -179,8 +179,14 @@ const action = async ({ form, cookies: reqCookies }: { form: Record<string, stri | ||
| 179 | 179 | </div> |
| 180 | 180 | |
| 181 | 181 | @if (!authed) |
| 182 | <div class="px-3 py-3 text-sm rounded-lg err-box">You need to be signed in to analyze a log entry.</div> | |
| 183 | <p class="mt-6 text-sm text-subtle"><StxLink to="/login" class="font-medium text-accent">Go to sign in</StxLink></p> | |
| 182 | {{-- Being signed out is not an error, so this is an empty state with an | |
| 183 | action rather than the red err-box it used to be. --}} | |
| 184 | <EmptyState | |
| 185 | title="Sign in required" | |
| 186 | description="You need to be signed in to analyze a log entry." | |
| 187 | > | |
| 188 | <StxLink to="/login" class="font-medium text-accent">Go to sign in</StxLink> | |
| 189 | </EmptyState> | |
| 184 | 190 | @elseif (!entry) |
| 185 | 191 | <h1 class="mb-2 font-semibold text-xl tracking-tight">Log entry not found</h1> |
| 186 | 192 | <p class="text-sm text-subtle">It may have been removed, or it belongs to an app you cannot access.</p> |
| @@ -94,10 +94,14 @@ const action = async ({ form, cookies: reqCookies }: { form: Record<string, stri | ||
| 94 | 94 | {{-- Reached without a session. The client guard this replaces navigated |
| 95 | 95 | away after hydration, so a logged-out visitor saw the form flash |
| 96 | 96 | first; this is the honest answer in the first response. --}} |
| 97 | <div class="px-3 py-3 text-sm rounded-lg err-box">You need to be signed in to create an app.</div> | |
| 98 | <p class="mt-6 text-center text-sm text-subtle"> | |
| 97 | {{-- Being signed out is not an error, so this is an empty state with an | |
| 98 | action rather than the red err-box it used to be. --}} | |
| 99 | <EmptyState | |
| 100 | title="Sign in required" | |
| 101 | description="You need to be signed in to create an app." | |
| 102 | > | |
| 99 | 103 | <StxLink to="/login" class="font-medium text-accent">Go to sign in</StxLink> |
| 100 | </p> | |
| 104 | </EmptyState> | |
| 101 | 105 | @else |
| 102 | 106 | {{-- Posts to itself; the action is in the server block. --}} |
| 103 | 107 | <form action="/projects/new" method="POST" class="space-y-4"> |