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

Student lists fixes #712 #713 #756

Merged
merged 1 commit into from
Dec 16, 2024
Merged

Student lists fixes #712 #713 #756

merged 1 commit into from
Dec 16, 2024

Conversation

hupf
Copy link
Collaborator

@hupf hupf commented Dec 16, 2024

Siehe #712 #713

Fach-/Klassenliste:

  • Wenn ich von "Aktuelle Fächer" auf die Fach-/Klassenliste gelange, wird in der Navigation unter "Unterricht" der Link "Tests und Bewertung" als aktiv angezeigt - falsche Verlinkung?
    • → Eigener PR
  • Die Namen sind falsch dargestellt: Vorname Nachname statt Nachname Vornamen. Damit stimmt auch die Sortierung (alphabetisch nach Nachname Vorname) nicht.
    Fachliste: ParticipatingStudents => FullName
    Klassenliste: /Persons => FullName
  • Klick auf den Namen sollte wohl am besten auf das Dossier > Adresse führen (nicht Absenzen; Spezifikation implizit falsch)
  • Layout: Die Anzahl Anmeldungen hat zu viel Abstand zur Bezeichnung Fach/Klasse => Bitte so wie bei den Tests.
  • Layout: Das Suchfeld hat zu viel Abstand zur Liste darunter => Bitte so wie bei der Präsenzkontrolle.
  • Breakpoints: Diese stimmen nicht exakt mit der Präsenzkontrolle überein, wenige Pixel Abweichung - weshalb ist das so? (oder bin ich zu pingelig ;)
  • Mail an alle: Kommatrennung funktioniert bei mir mit Outlook nicht, wird als 1 E-Mail-Adresse angeschaut => Bitte mit Semikolon trennen.
  • Klassenliste: Der Lehrbetrieb wird nicht angezeigt.

Studengangsliste:

  • Platzhalter im Suchfeld falsch: Richtig ist "Name oder Status suchen" (vgl. Mockup)
  • Gleiches Problem wie bei
    Anlassdetails: Fach- und Klassenliste - 3PT #712 : Sortierung falsch (Vorname Nachname) => Sortierung bitte nach FullName (Nachname Vornamen)
  • Richtung des Pfeils bei Sortierung vollständiger Name: A-Z (aufsteigend) => Pfeil nach oben
    image

@hupf hupf self-assigned this Dec 16, 2024
@hupf hupf changed the title Student lists fixes #712 #713 Draft: Student lists fixes #712 #713 Dec 16, 2024
@hupf hupf force-pushed the bug/712-713-fixes branch 2 times, most recently from d8a6bb9 to 9834b9b Compare December 16, 2024 12:37
@hupf hupf changed the title Draft: Student lists fixes #712 #713 Student lists fixes #712 #713 Dec 16, 2024
@hupf hupf force-pushed the bug/712-713-fixes branch from 9834b9b to 9b844c2 Compare December 16, 2024 12:45
- Display & sort by lastname, firstname
- Remove heading margin
- Adjust margin after header
- Use ';' as emails separator in mailto link
- Show companies on class list as well
- Rename search placeholder
- Invert study course list sort arrow
- Visit dossier addresses instead of absences tab
- Switch presence control grid from breakpoints to CSS grid
@hupf hupf force-pushed the bug/712-713-fixes branch from 9b844c2 to 23f3623 Compare December 16, 2024 12:46
@hupf hupf merged commit 5d356d9 into main Dec 16, 2024
2 checks passed
@hupf hupf deleted the bug/712-713-fixes branch December 16, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant