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/336 improve points edit #375

Merged
merged 3 commits into from
Apr 20, 2022
Merged

Conversation

mburri
Copy link
Contributor

@mburri mburri commented Apr 19, 2022

Es gab noch drei Unschönheiten bei der Eingabe von Punkten:

  • Fokus ging verloren nachdem ein Wert eingegeben wurde - dies kann mit den trackBy Properties auf den ngFor behoben werden. Für verschachtelte ngFors muss dies für jede Stufe implementiert werden.
  • Mit Tab konnte nicht durch die Tabelle navigiert werden - bzw. die Navigation erfolgte pro Zeile, das ist aber meistens nicht was man will - man möchte eher durch die Spalte navigieren können.
  • In der Mobile View wurde das Eingebafeld nicht angzeigt, wenn noch kein Resultat existiert - v.a. weil der boolsche Vergleich einen Fehler enthielt - das fixen/ vereinfachen des Ausdrucks hat dann auch gleich das Problem behoben.

@mburri mburri requested a review from mfehlmann April 19, 2022 19:01
@mfehlmann mfehlmann merged commit 27685f1 into master Apr 20, 2022
@mfehlmann mfehlmann deleted the feature/336-improve-points-edit branch April 20, 2022 08:38
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.

2 participants