Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Präsenzkontrolle: Dropdown mit allen Lektionen des Tages (1PT) #145

Closed
fbufbi opened this issue Feb 24, 2020 · 8 comments
Closed

Präsenzkontrolle: Dropdown mit allen Lektionen des Tages (1PT) #145

fbufbi opened this issue Feb 24, 2020 · 8 comments
Assignees
Labels
module-Präsenzkontrolle Feature fürs Modul "Präsenzkontrolle"

Comments

@fbufbi
Copy link
Collaborator

fbufbi commented Feb 24, 2020

In der Präsenzkontrolle soll es einfacher möglich sein, zu jeder eigenen Lektion des Tages zu springen. Bei Fach und Klasse soll hierzu ein Dropdown mit allen eigenen Fächern des Tages zur Verfügung stehen.

  • Die in der Präsenzkontrolle (im Hintergrund) angezeigte Lektion ist im Dropdown grau hinterlegt
  • Das Dropdown ist immer vorhanden. Wenn ich als Lehrkraft nur 1 Lektion an diesem Tag habe, ist diese im Dropdown drin
  • Zwei Zeilen, nicht allzu breit
  • In der Tablet und Desktop-Ansicht nimmt das Dropdown nicht die gesamte Breite ein

image

@fbufbi fbufbi added the module-Präsenzkontrolle Feature fürs Modul "Präsenzkontrolle" label Feb 24, 2020
@fbufbi fbufbi added this to the 1_Präsenzkontrolle milestone Feb 24, 2020
@hupf hupf added konzeption Task für Konzeption (POs & UX) and removed konzeption Task für Konzeption (POs & UX) labels Apr 20, 2020
@lmathys lmathys changed the title Präsenzkontrolle: Dropdown mit allen Lektionen des Tages Präsenzkontrolle: Dropdown mit allen Lektionen des Tages (1PT) Apr 21, 2020
@pmurkowsky
Copy link

@fbufbi Hier ein Vorschlag zum Review.

Lektionen-Overlay

@fbufbi
Copy link
Collaborator Author

fbufbi commented Apr 24, 2020

@pmurkowsky Danke für den Vorschlag, ist angenommen.

Bemerkungen und Fragen:

  • Es soll nur 2 Zeilen im Dropdown geben. Bei den Berufsfachschulen haben wir nur die Fachbezeichnung und die Klasse, z.B. "Gesellschaft, DHF2018a". Die sog. Namenskonvention der Gymnasien ist (noch) etwas komplizierter, soll aber nicht die drei Zeilen verursachen. Ergo:
    Zeile 1: LessonPresences.LessonDateTimeFrom-LessonPresences.LessonDateTimeTo
    Zeile 2: LessonPresences.EventDesignation, LessonPresences.StudyClassNumber
  • Auf Tablet und Desktop soll das Dropdown nicht die gesamte Breite des Bildschirms einnehmen, sondern nur eine gewisse Breite haben (fix oder so breit wie der längste Eintrag)
  • Was machen wir, wenn keine weitere Lektion vorhanden ist? -> Pfeil nach unten ist disabled und entsprechende Notifikation beim Draufdrücken?
  • Was heisst "aktuelle Lektion ist grau hinterlegt"? Aktuell gemäss Uhrzeit oder aktuell ausgewählt?

@pmurkowsky
Copy link

* Es soll nur 2 Zeilen im Dropdown geben. Bei den Berufsfachschulen haben wir nur die Fachbezeichnung und die Klasse, z.B. "Gesellschaft, DHF2018a". Die sog. Namenskonvention der Gymnasien ist (noch) etwas komplizierter, soll aber nicht die drei Zeilen verursachen. Ergo:
  Zeile 1: `LessonPresences.LessonDateTimeFrom`-`LessonPresences.LessonDateTimeTo`
  Zeile 2: `LessonPresences.EventDesignation`, `LessonPresences.StudyClassNumber`

Die Argumentation war die höhere Übersichtlichkeit bei 3 Zeilen. Die Klasse ist so einfacher lesbar. Alternativ könnte man die Klasse auch so darstellen.

Lektionen-Overlay-2-Zeilen

* Auf Tablet und Desktop soll das Dropdown nicht die gesamte Breite des Bildschirms einnehmen, sondern nur eine gewisse Breite haben (fix oder so breit wie der längste Eintrag)

Einverstanden. Etwa so?

Lektionen-Overlay-Desktop

* Was machen wir, wenn keine weitere Lektion vorhanden ist? -> Pfeil nach unten ist disabled und entsprechende Notifikation beim Draufdrücken?

Nein, es werden immer alle Lektionen des Tages angezeigt, auch wenn ich in der letzten Stunde bin.

* Was heisst "aktuelle Lektion ist grau hinterlegt"? Aktuell gemäss Uhrzeit oder aktuell ausgewählt?

Gute Frage. Wahrscheinlich aktuell ausgewählt.

@fbufbi fbufbi removed the konzeption Task für Konzeption (POs & UX) label May 1, 2020
@caebr caebr assigned caebr and hupf and unassigned caebr May 6, 2020
@hupf hupf assigned caebr and unassigned hupf May 13, 2020
@caebr caebr assigned hupf and unassigned caebr May 13, 2020
@hupf hupf assigned schefbi and fbufbi and unassigned hupf May 18, 2020
@schefbi
Copy link
Contributor

schefbi commented May 19, 2020

@hupf Wenn auf Button geklickt wird, wird die Seite neu geladen. type="button" einfügen weil Evento in einem form Tag läuft.

@hupf
Copy link
Collaborator

hupf commented May 19, 2020

@schefbi Danke für den Hinweis, habe den Typ noch hinzugefügt und #178 erstellt, dass wir dass in Zukunft allenfalls linten könnten.

hupf added a commit that referenced this issue May 19, 2020
@fbufbi
Copy link
Collaborator Author

fbufbi commented May 20, 2020

@hupf

  1. Wenn das Dropdown im mobilen Modus den ganzen Bildschirm breit ist, dann wird die oberste Linie nicht angezeigt (oberer Pfeil).
  2. Das Grau beim Scrollen durch die verschiedenen Lektionen (unterer Pfeil) ist zu hell und kaum sichtbar. Liegt das an unserem CSS?
    image

@hupf
Copy link
Collaborator

hupf commented May 20, 2020

@fbufbi Danke für deine Rückmeldung.

  1. Die oberste Linie wird nun auch kleinen Screens angezeigt.
  2. Die Hover-Hintergrundfarbe war auf der Standardeinstellung von Boostrap. Ziel davon ist schon, dass es eher ein subtiler Effekt ist. Ich habe die Farbe nun mal auf das nächst dunklere Grau gestellt. Findest du es nun besser so?

image

hupf added a commit that referenced this issue May 20, 2020
@fbufbi
Copy link
Collaborator Author

fbufbi commented May 20, 2020

Test OK

@fbufbi fbufbi closed this as completed May 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module-Präsenzkontrolle Feature fürs Modul "Präsenzkontrolle"
Projects
None yet
Development

No branches or pull requests

5 participants