search

HBTGmbH / salat

5 stars 1 forks source link

Fehlenden Arbeitsbeginn/Pausendauer in Matrixübersicht kennzeichnen #359

Closed cghbt closed 2 months ago

cghbt commented 3 months ago

Da wir ja jetzt immer den Arbeitsbeginn und die Pausendauer buchen müssen, wäre es gut, wenn der Salat anzeigen würde, ob das für alle Tage vorhanden ist. Man könnte bspw. in der Matrixübersicht kennzeichnen, wenn es Tage mit Buchungen aber ohne diese Angaben gibt. Diese Tage könnte man mit einem entsprechenden Icon oder farbig kennzeichnen.

Aktuell kann man das nur etwas mühsam über die Einzelübersicht prüfen, indem man den ganzen Monat Tag für Tag durchklickt.

hehu80 commented 2 months ago

Was hältst du vom Anzeigen der Anfangs-/Pausenzeiten als ersten Wurf? image

cghbt commented 2 months ago

Ja, sieht schon ganz gut aus! Man könnte das vielleicht noch Farblich oder mit Symbolen kennzeichnen, so wie Adrian auch vorgeschlagen hat. Also vielleicht, Spalten mit Buchungen aber ohne Arbeitsbeginn/Pausendauer rot einfärben. Oder da in die unteren Zeilen rote "X"-Icons reinmachen.

hehu80 commented 2 months ago

Ein passendes Symbol hab ich gerade nicht. Wie ist das? image

cghbt commented 2 months ago

Ja, gut. Nur das Rot ist sehr Primärfarbe, das tut etwas in den Augen weh. Ich würde ein etwas sanfteren Farbton nehmen, so in diese Richtung: https://imagecolorpicker.com/color-code/e05b50

VG Claas

hehu80 commented 2 months ago

Eine Farbe, die Mandy heißt, nehme ich natürlich sofort :smile: image

cghbt commented 2 months ago

Funktioniert generell gut, vielen Dank! Aber es sind nicht alle Fälle angedeckt. Wenn man nur max. 6 Stunden arbeitet, muss man ja keine Pause machen. Dann wird es trotzdem rot. Und wenn man krank war oder Urlaub hatte auch. Aber als MVP schon mal okay und deutlich besser als vorher.

hehu80 commented 2 months ago

Die Prüfung, ob die Länge der Pause korrekt ist, habe ich bewusst ausgeklammert, weil @Peglow-C damit schon angefangen hatte (https://github.com/HBTGmbH/salat/issues/354). Seine Änderungen könnte man jetzt bestimmt auch in der Matrixübersicht nutzen. Gibt auch schon ein neues Issue dazu: https://github.com/HBTGmbH/salat/issues/368