Ändere Wochenfeld von Auswahl "Jede/gerade Vorlesungswochen/ungerade Vorlesungswochen" zu Freitextfeld mit Vorschlag. #37

Merged
Gonne merged 1 commit from wochen into main 2025-04-03 07:42:09 +00:00
Owner

Closes #28

Closes #28
Gonne added 1 commit 2025-04-01 17:23:25 +00:00
requested review from johannes 2025-04-01 17:31:24 +00:00
johannes requested changes 2025-04-01 19:58:26 +00:00
@ -65,2 +61,2 @@
</select>
<label for="woche">Vorlesungswoche</label>
<input class="form-control form-text" type="text" name="woche" id="woche" list="woche_list" value="{{$.Date.Week}}">
<datalist id="woche_list">
Member

datalists werden von Browsern nur eingeschränkt unterstützt, könnte Probleme in Safari geben und wird auf jeden Fall nicht in Firefox Android funktionieren: https://caniuse.com/datalist

Ich erwarte, dass einige Menschen das am Handy nutzen und würde daher lieber ein besser unterstütztes HTML-Element wählen. Ein Freitextfeld ist mir hier auch gar nicht so wichtig, können wir nicht einfach ein Select draus machen, vielleicht mit einer vierten Option "Sonstiges" und man soll dann das Sonstiges-Raum-Feld für Bemerkungen nutzen (ist ja schon ein Freitextfeld, könnten seine intendierte Benutzung verallgemeinern).

datalists werden von Browsern nur eingeschränkt unterstützt, könnte Probleme in Safari geben und wird auf jeden Fall nicht in Firefox Android funktionieren: https://caniuse.com/datalist Ich erwarte, dass einige Menschen das am Handy nutzen und würde daher lieber ein besser unterstütztes HTML-Element wählen. Ein Freitextfeld ist mir hier auch gar nicht so wichtig, können wir nicht einfach ein Select draus machen, vielleicht mit einer vierten Option "Sonstiges" und man soll dann das Sonstiges-Raum-Feld für Bemerkungen nutzen (ist ja schon ein Freitextfeld, könnten seine intendierte Benutzung verallgemeinern).
Author
Owner

Soweit ich die Kompatiblität lese, wäre es für Menschen ohne Unterstützung ein Freitextfeld ohne Autovervollständigung. Das fände ich vertretbar.

Soweit ich die Kompatiblität lese, wäre es für Menschen ohne Unterstützung ein Freitextfeld ohne Autovervollständigung. Das fände ich vertretbar.
Gonne marked this conversation as resolved
@ -66,1 +61,3 @@
<label for="woche">Vorlesungswoche</label>
<input class="form-control form-text" type="text" name="woche" id="woche" list="woche_list" value="{{$.Date.Week}}">
<datalist id="woche_list">
<option>Jede Woche</option>
Member

Fände es eigentlich besser, dass diese Option mit dem leeren String als Wert gespeichert wird (und in der Tabelle dann keinen Platz verbraucht)

Fände es eigentlich besser, dass diese Option mit dem leeren String als Wert gespeichert wird (und in der Tabelle dann keinen Platz verbraucht)
Author
Owner

Sehe ich ein. Damit das passiert, brauchen wir irgendeine Form von Auswahl. Also probiere ich mich nachher an Radio Buttons.

Sehe ich ein. Damit das passiert, brauchen wir irgendeine Form von Auswahl. Also probiere ich mich nachher an Radio Buttons.
Gonne marked this conversation as resolved
Gonne force-pushed wochen from deced3e596 to e05d30b886 2025-04-02 12:52:26 +00:00 Compare
requested review from johannes 2025-04-02 12:52:39 +00:00
johannes reviewed 2025-04-02 15:12:07 +00:00
@ -65,0 +63,4 @@
<option value="1"{{if eq 1 $.Date.Week}} selected{{end}}>In Wochen mit Übung</option>
<option value="2"{{if eq 2 $.Date.Week}} selected{{end}}>In Wochen ohne Übung</option>
<option value="1"{{if eq 3 $.Date.Week}} selected{{end}}>In geraden Vorlesungswochen</option>
<option value="2"{{if eq 4 $.Date.Week}} selected{{end}}>In ungeraden Vorlesungswochen</option>
Member

Ich werde dich nicht aufhalten, aber bin der Meinung, dass diese beiden Optionen unnötiger Overhead sind und man sie weglassen kann.

Ich werde dich nicht aufhalten, aber bin der Meinung, dass diese beiden Optionen unnötiger Overhead sind und man sie weglassen kann.
Author
Owner

Vermutlich. Ich will es mal ein Semester ausprobieren.

Vermutlich. Ich will es mal ein Semester ausprobieren.
Gonne merged commit e05d30b886 into main 2025-04-03 07:42:09 +00:00
Gonne deleted branch wochen 2025-04-03 07:42:09 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Fachschaft/sprechstunden-go#37
No description provided.