Skip to content

[WIP] Show basic Calendar on Dashboard#212

Draft
JohannHoepfner wants to merge 2 commits intomainfrom
calendar-ui
Draft

[WIP] Show basic Calendar on Dashboard#212
JohannHoepfner wants to merge 2 commits intomainfrom
calendar-ui

Conversation

@JohannHoepfner
Copy link
Copy Markdown
Member

No description provided.

@JohannHoepfner JohannHoepfner added javascript Pull requests that update javascript code .NET Pull requests that update .NET code labels Feb 21, 2026
@JohannHoepfner JohannHoepfner changed the title WIP: Show basic Calendar on Dashboard [WIP] Show basic Calendar on Dashboard Mar 3, 2026
@JohannHoepfner JohannHoepfner force-pushed the main branch 2 times, most recently from 90a9b18 to ecd400c Compare March 10, 2026 07:41
Copy link
Copy Markdown
Member

@rschlossarek rschlossarek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rein funktionale Kommentare ohne den Code gelesen zu haben:

  • Das Design passt nicht zum Rest der App
  • Mein Browser ist auf Englisch konfiguriert. Da wir aber sonst die Locale fest auf Deutsch haben, ist der Kalender jetzt bei mir das einzige, was Englisch erscheint
  • Die Schrift auf dem Blau erfüllt die Anforderungen an Kontraste für Barrierefreiheit nicht
  • Es wäre cool, wenn ich auf die Otiums-Angebote klicken könnte, und dann auf die entsprechenden Seiten käme (wie das in der Tabelle schon geht)
  • Eine farbliche Unterscheidbarkeit von Otium und Profundum oder evtl. sogar der Profundums-Kategorie wäre cool
  • Wir sollten die Räume mit anzeigen
  • Wir könnten nicht belegte Otiumsslots als Umriss mit darstellen
  • Die Ladezeit von 1/4 Sekunde scheint mir etwas enorm viel. Ggf. nicht alle Wochen laden?
  • Bei Lehrer:innen ggf. die Belegungen der Mentees mit anzeigen?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update javascript code .NET Pull requests that update .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants