feat: set default expiration in select #28

tapiron · 2025-09-12T19:09:35+02:00

tapiron commented

2025-09-12 19:09:35 +02:00

No default value in select could be interpreted as ever lasting note.
Default value in the select makes that clearer.

No default value in select could be interpreted as ever lasting note. Default value in the select makes that clearer.

tapiron added 1 commit 2025-09-12 19:09:36 +02:00

feat: set default expiration in select 16945d488a

jriou requested changes 2025-09-13 19:18:16 +02:00

src/server/templates/index.html

					
				@ -37,3 +37,3 @@

				                <div class="col">

				                    <select class="form-select" aria-label="Expiration" id="expiration" name="expiration">

				                        <option selected="selected" disabled>Expiration</option>

				                        <option selected="selected" disabled>{{HumanDuration .Expiration}}</option>

jriou commented

2025-09-13 19:18:08 +02:00

Expiration is included in Expirations resulting in a duplicate entry in the list.

Wouldn't it be better to use a selected="selected" attribute when the expiration is the default one instead?

Something like:

<option disabled>Expiration</option>
{{range .Expirations}}
 <option {{if eq . .Expiration}}selected="selected"{{end}} value="{{.}}">{{HumanDuration .}}</option>
{{end}}

I'm not sure about the scope of the variables here but that should be doable. That way, we'll still see it's an expiration thanks to the disabled option but it would automatically select the default value set on the server.

Expiration is included in Expirations resulting in a duplicate entry in the list. Wouldn't it be better to use a `selected="selected"` attribute when the expiration is the default one instead? Something like: ```html <option disabled>Expiration</option> {{range .Expirations}} <option {{if eq . .Expiration}}selected="selected"{{end}} value="{{.}}">{{HumanDuration .}}</option> {{end}} ``` I'm not sure about the scope of the variables here but that should be doable. That way, we'll still see it's an expiration thanks to the disabled option but it would automatically select the default value set on the server.