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

Feature/333 bugfixes #385

Merged
merged 11 commits into from
Apr 29, 2022
Merged

Feature/333 bugfixes #385

merged 11 commits into from
Apr 29, 2022

Conversation

mburri
Copy link
Contributor

@mburri mburri commented Apr 27, 2022

Die am Daily besprochenen Bugfixes und eine kleine Anpassung:

  • Klasse fehlt im Header (Seite Testübersicht)
  • Mittelwerte sollten nie "0" sein - ich habe jetzt einfach den Wert ganz weg gelassen - als Vorschlag
  • Tab funktioniert jetzt auch für die Dropdown - interessanterweise kann man zwei Elementen den gleichen Tabindex geben, dann springt es vom 1ten zum 2ten - und weil Dropdowns die disabled sind übersprungen werden, funktionert alles tiptop.
  • Beim Scrollen seitwärts ändert sich die 3. Spalte jetzt nicht mehr - allerdings ist die gesamte Umsetzung etwas fragil - vielleicht hat @hupf noch eine Idee wie man das besser machen könnte

Nicht umgesetzt ist das "Problem", dass man die Note nicht löschen kann, weil das Backend das offenbar gar nicht zulässt - bzw. Sandro ist das noch am abklären - ich habe eine Story geschrieben und ans Board gehängt.

Zusätzlich habe ich noch folgende Anpassungen gemacht:

  • der Disabled State des Dropdowns ist jetzt über ein Subject/ Observable gelöst
  • Das Todo in course.model.ts konnte ich erledigen
  • Damit nicht zuviele Requests abgesetzt werden, wenn man im Dropdown eine Note per Tastatur auswählt, ohne das Dropdown an sich zu öffnen habe ich da auch noch ein Debouncing eingeführt.

@mburri mburri requested a review from caebr April 27, 2022 12:30
@caebr
Copy link
Collaborator

caebr commented Apr 29, 2022

@mburri Bei den Mittelwerten wird weiterhin "0" anzgezeigt

Copy link
Collaborator

@caebr caebr left a comment

Choose a reason for hiding this comment

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

Mittelwert 0 noch anpassen, sonst ok

@mburri mburri requested a review from caebr April 29, 2022 09:28
@mburri mburri merged commit 26efad1 into master Apr 29, 2022
@mburri mburri deleted the feature/333-bugfixes branch April 29, 2022 10:06
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.

3 participants