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

Konsistentes Farbschema der buttons #1238

Closed
ManuelMoeri opened this issue Dec 9, 2024 · 2 comments · Fixed by #1315
Closed

Konsistentes Farbschema der buttons #1238

ManuelMoeri opened this issue Dec 9, 2024 · 2 comments · Fixed by #1315
Assignees
Labels
bug Something isn't working UX UX is involved

Comments

@ManuelMoeri
Copy link
Collaborator

ManuelMoeri commented Dec 9, 2024

Describe the bug
Der bug tritt bei dem 'Speichern' button einiger Dialog auf. Diese buttons haben das falsche Farbschema.

To Reproduce
Steps to reproduce the behavior:

  1. Besuche pitc.okr.localhost:4200 oder acme.okr.localhost:4200
  2. Drücke auf das Drei Punkte menü eines beliebigen Objectives.
  3. Wähle die Option 'Objective Duplizieren aus.
  4. Der 'Speichern' button auf dem Dialog erscheint in der secondary anstatt der primary Farbe.

Expected behavior
Buttons die als Haupt-action dienen sollten immer die primary Farbe erhalten. -> Mit UX klären
Dies soll über das ganze Tool hingweg konsistent sein -> Checken und allenfalls beheben ob dies noch anderen orten auftritt.

Screenshots
Falsch
Image

Richtig
Image

Additional context
Ob wirklich jeder Speichern button primary colors erhalten soll, sollte noch mit UX geklärt werden. Meines wissens nach schon.

@ManuelMoeri ManuelMoeri added the bug Something isn't working label Dec 9, 2024
@ManuelMoeri ManuelMoeri assigned peggimann and unassigned peggimann Dec 9, 2024
@ManuelMoeri ManuelMoeri added the UX UX is involved label Dec 9, 2024
@ManuelMoeri ManuelMoeri self-assigned this Jan 15, 2025
@ManuelMoeri
Copy link
Collaborator Author

ManuelMoeri commented Jan 15, 2025

15.01.2025

ToDo

  • Ticket lesen
  • Applikation nach fehlern durchkämmen, hier Dokumentieren & dann beheben.
    • Objective duplizieren - Speichern
    • Objective bearbeiten - Speichern
    • Check-in erfassen - Action Speichern
  • Unklarheiten mit UX absprechen
    • Check-in erfassen - Action speichern wird noch abgeklärt. -> Wurde abgeklärt ein Ticket zum evauluieren einer anderen Lösung wurde im Backlog erfasst.
  • Tests schreiben
  • Smoke tests
  • ✔️

@ManuelMoeri ManuelMoeri linked a pull request Jan 17, 2025 that will close this issue
@ManuelMoeri
Copy link
Collaborator Author

ManuelMoeri commented Jan 20, 2025

Stand 20.01.2025

Heute habe ich nebst den Meetings weiter daran gearbeitet die Tests zu schreiben. Bei diesen habe ich die E2E Tests für alle KeyResult und Objective dialoge beendet. Was noch zu tun ist, sind die Dialoge rund um die Check-ins und sämtliche Dialoge bezüglich der Teamverwaltung. Um einfacher auf primary buttons zu überprüfen habe ich eine funktion namens checkForPrimaryButton implementiert. Diese ist zum teil in den verschiedenen Dialogen mit unterschiedlichen test-ids zum klicken implementiert aber im Grundsatz gleich.

TODO

  • E2E Tests Check-ins
  • E2E Tests Teamverwaltung
  • Cleanup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working UX UX is involved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants