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.
| @@ -211,7 +211,7 @@ const action = async ({ form, cookies: reqCookies }: { form: Record<string, stri | ||
| 211 | 211 | </section> |
| 212 | 212 | |
| 213 | 213 | @if (formError) |
| 214 | <div class="mb-6 px-3 py-2 text-sm rounded-lg err-box">{{ formError }}</div> | |
| 214 | <div role="alert" class="mb-6 px-3 py-2 text-sm rounded-lg err-box">{{ formError }}</div> | |
| 215 | 215 | @endif |
| 216 | 216 | |
| 217 | 217 | @if (!enabled) |
| @@ -102,12 +102,12 @@ const action = async ({ form, cookies: reqCookies }: { form: Record<string, stri | ||
| 102 | 102 | {{-- Posts to itself; the action is in the server block. --}} |
| 103 | 103 | <form action="/projects/new" method="POST" class="space-y-4"> |
| 104 | 104 | @if (formError) |
| 105 | <div class="px-3 py-2 text-sm rounded-lg err-box">{{ formError }}</div> | |
| 105 | <div id="form-error" role="alert" class="px-3 py-2 text-sm rounded-lg err-box">{{ formError }}</div> | |
| 106 | 106 | @endif |
| 107 | 107 | |
| 108 | 108 | <div> |
| 109 | 109 | <label for="newapp-name" class="block mb-1.5 font-medium text-muted text-xs">App name</label> |
| 110 | <input id="newapp-name" name="name" type="text" required autofocus placeholder="Acme Web" value="" class="px-3.5 py-2.5 w-full text-sm field"> | |
| 110 | <input id="newapp-name" name="name" type="text" required autofocus placeholder="Acme Web" value="" class="px-3.5 py-2.5 w-full text-sm field" aria-invalid="false"@if (formError) aria-describedby="form-error"@endif> | |
| 111 | 111 | </div> |
| 112 | 112 | <div> |
| 113 | 113 | <label for="newapp-platform" class="block mb-1.5 font-medium text-muted text-xs">Platform</label> |