ClockGU / clock-frontend

https://clock.uni-frankfurt.de
GNU Affero General Public License v3.0
0 stars 0 forks source link

Kalender Month Switcher #279

Closed sheepyhollow closed 1 year ago

sheepyhollow commented 2 years ago

Der Month Switcher im Kalender sollte so aussehen wie in den anderen Komponenten. Das sollte durch eine Bearbeitung von CalendarNavigationButtons.vue möglich sein.

sheepyhollow commented 2 years ago

Allerdings sehe ich gerade nicht, wo der eigentliche Monat in der Anzeige herkommt 🤔

sheepyhollow commented 2 years ago

Ich sehe nicht so richtig den Sinn in der Komponente CalendarNavigationButtons.vue.

Die geringe Funktionalität rechtfertigt eigentlich keine eigene Komponente - ich würde vorschlagen, alle Buttons direkt in Calendar.vue einzubauen, dann sollte sich das oben auch einigermaßen lösen lassen.

Tatsächlich wäre es noch konsistenter mit den anderen Views, den MonthSwitcher zu verwenden - es kann aber sein, dass nicht so einfach ist weil die v-calender component schon recht viel tut.

sheepyhollow commented 2 years ago

Allerdings sehe ich gerade nicht, wo der eigentliche Monat in der Anzeige herkommt

Das ist der $refs.calendar.title

sheepyhollow commented 2 years ago

Tatsächlich ist es möglich, im Kalender beliebig weit vorwärts und rückwärts zu blättern... Alle anderen Komponenten sind limitiert auf die Range des aktuellen Vertrags.

in ShiftsView wird der DataFilter benutzt, um diese ganzen Informationen zu erfassen. Möglicherweise ist das für den Kalender auch möglich und Voraussetzung für die Verwendung des MonthSwitcher. Allerdings mag das Verdrahten wie gesagt schwierig sein.

chgad commented 1 year ago

Um das Aufzugreifen:

Limitieren wir die Ansicht auf den jeweils ausgewählten Vertrag, so geht dem Nutzer die gesamt Übersicht verloren.

Lassen wir den Nutzern die möglichkeit den Kalender nach Lust und Laune zu durchblättern, so würden sie spätestens in Kombination mit #453 sehen "Oh, mein ausgewählter Vertrag ist hier gar nicht gültig." .

Am Ende ist es eine Frage von "Übersicht vs. ShiftForm invalid für Dataum".

Da Punar auch #453 angeregt hat solltest du sie mal fragen, wie sie diesen Kontext bewertet.

sheepyhollow commented 1 year ago

Ich verlinke hier auch nochmal #182 - das ist zwar unser ältestes, noch lebendes Issue und ein eigenes großes Brett, aber sollte im Hinterkopf bleiben: Wenn wir einen Kalender hätten, der die Schichten aller Verträge zeigt, dann wäre ein Datumslimit erst recht Unsinn.

Ich glaube aber, dass der Vuetify2-Kalender nicht soo flexibel ist. In Vuetify3 gibt es auch gar keinen Kalender im Angebot...