This thread contains a patchset. You're looking at the original emails,
but you may wish to use the patch review UI.
Review patch
1
[PATCH] Fix spacing for EXPIRY in create form.
Fixes: https://todo.sr.ht/~brettgilio/snippet/1
---
ui/html/pages/create.gohtml | 68 ++++++++++++++++++-------------------
1 file changed, 34 insertions(+), 34 deletions(-)
diff --git a/ui/html/pages/create.gohtml b/ui/html/pages/create.gohtml
index 9768fa9..2f67c0f 100644
--- a/ui/html/pages/create.gohtml
+++ b/ui/html/pages/create.gohtml
@@ -78,46 +78,46 @@
{{end}}
</div>
- <div class="grid grid-cols-4 gap-4 text-center">
+ <div class="flex flex-row flex-wrap items-center gap-2 text-center">
<div>
<p class="flex items-center justify-between pr-4 py-4 text-sm font-medium dark:text-white">
<span>Expires in: </span>
</p>
</div>
- <div>
- <input class="hidden peer" type="radio" name="expires" value="365" id="oneyear" {{if (eq
- .Form.ExpiresInDays 365)}}checked{{end}} />
-
- <label
- class="flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
- for="oneyear">
- <span> One Year </span>
- </label>
- </div>
-
- <div>
- <input class="hidden peer" type="radio" name="expires" value="7" id="oneweek" {{if (eq
- .Form.ExpiresInDays 7)}}checked{{end}} />
-
- <label
- class="flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
- for="oneweek">
- <span> One Week </span>
- </label>
- </div>
-
- <div>
- <input class="hidden peer" type="radio" name="expires" value="1" id="oneday" {{if (eq
- .Form.ExpiresInDays 1)}}checked{{end}} />
-
- <label
- class="flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
- for="oneday">
- <span> One Day </span>
- </label>
- </div>
-
+ <div class="flex flex-1 flex-row gap-4 items-stretch">
+ <div class="flex-1">
+ <input class="hidden peer" type="radio" name="expires" value="365" id="oneyear"
+ {{if (eq .Form.ExpiresInDays 365)}}checked{{end}} />
+
+ <label
+ class="h-full flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
+ for="oneyear">
+ <span> One Year </span>
+ </label>
+ </div>
+
+ <div class="flex-1">
+ <input class="hidden peer" type="radio" name="expires" value="7" id="oneweek"
+ {{if (eq .Form.ExpiresInDays 7)}}checked{{end}} />
+
+ <label
+ class="h-full flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
+ for="oneweek">
+ <span> One Week </span>
+ </label>
+ </div>
+
+ <div class="flex-1">
+ <input class="hidden peer" type="radio" name="expires" value="1" id="oneday"
+ {{if (eq .Form.ExpiresInDays 1)}}checked{{end}} />
+ <label
+ class="h-full flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
+ for="oneday">
+ <span> One Day </span>
+ </label>
+ </div>
+ </div>
</div>
<div>
--
2.37.1
On 7/28/2022 7:33 AM, Alexandru-Sergiu Marton wrote:
> Fixes: https://todo.sr.ht/~brettgilio/snippet/1
> ---
> ui/html/pages/create.gohtml | 68 ++++++++++++++++++-------------------
> 1 file changed, 34 insertions(+), 34 deletions(-)
>
> diff --git a/ui/html/pages/create.gohtml b/ui/html/pages/create.gohtml
> index 9768fa9..2f67c0f 100644
> --- a/ui/html/pages/create.gohtml
> +++ b/ui/html/pages/create.gohtml
> @@ -78,46 +78,46 @@
> {{end}}
> </div>
>
> - <div class="grid grid-cols-4 gap-4 text-center">
> + <div class="flex flex-row flex-wrap items-center gap-2 text-center">
> <div>
> <p class="flex items-center justify-between pr-4 py-4 text-sm font-medium dark:text-white">
> <span>Expires in: </span>
> </p>
> </div>
>
> - <div>
> - <input class="hidden peer" type="radio" name="expires" value="365" id="oneyear" {{if (eq
> - .Form.ExpiresInDays 365)}}checked{{end}} />
> -
> - <label
> - class="flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
> - for="oneyear">
> - <span> One Year </span>
> - </label>
> - </div>
> -
> - <div>
> - <input class="hidden peer" type="radio" name="expires" value="7" id="oneweek" {{if (eq
> - .Form.ExpiresInDays 7)}}checked{{end}} />
> -
> - <label
> - class="flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
> - for="oneweek">
> - <span> One Week </span>
> - </label>
> - </div>
> -
> - <div>
> - <input class="hidden peer" type="radio" name="expires" value="1" id="oneday" {{if (eq
> - .Form.ExpiresInDays 1)}}checked{{end}} />
> -
> - <label
> - class="flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
> - for="oneday">
> - <span> One Day </span>
> - </label>
> - </div>
> -
> + <div class="flex flex-1 flex-row gap-4 items-stretch">
> + <div class="flex-1">
> + <input class="hidden peer" type="radio" name="expires" value="365" id="oneyear"
> + {{if (eq .Form.ExpiresInDays 365)}}checked{{end}} />
> +
> + <label
> + class="h-full flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
> + for="oneyear">
> + <span> One Year </span>
> + </label>
> + </div>
> +
> + <div class="flex-1">
> + <input class="hidden peer" type="radio" name="expires" value="7" id="oneweek"
> + {{if (eq .Form.ExpiresInDays 7)}}checked{{end}} />
> +
> + <label
> + class="h-full flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
> + for="oneweek">
> + <span> One Week </span>
> + </label>
> + </div>
> +
> + <div class="flex-1">
> + <input class="hidden peer" type="radio" name="expires" value="1" id="oneday"
> + {{if (eq .Form.ExpiresInDays 1)}}checked{{end}} />
> + <label
> + class="h-full flex items-center justify-center p-4 text-sm font-medium transition-colors border border-gray-100 rounded-lg shadow-sm cursor-pointer peer-checked:border-blue-500 hover:bg-blue-500/10 peer-checked:ring-1 peer-checked:ring-blue-500"
> + for="oneday">
> + <span> One Day </span>
> + </label>
> + </div>
> + </div>
> </div>
>
> <div>
Thank you for sending this. I am going to hang on to this until the
create UI stabilizes a bit further.