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

Dialoge auf mobile #624

Closed
3 tasks done
peggimann opened this issue Nov 27, 2023 · 2 comments · Fixed by #1072
Closed
3 tasks done

Dialoge auf mobile #624

peggimann opened this issue Nov 27, 2023 · 2 comments · Fixed by #1072
Assignees
Labels
bug Something isn't working

Comments

@peggimann
Copy link
Collaborator

peggimann commented Nov 27, 2023

Wenn ein Dialog auf mobile Ansicht geöffnet wird ist der Abstand zum Rand zu hoch.
Die Dialoge haben neu im CSS eine max-height gesetzt bekommen. @lkleisa hat eine Umsetzung vorher gemacht, wie es ohne funktioniert hat.

Es gibt bereits einen PR, diesen zuerst ansehen und wenn nötig erweitern

Anforderungen

  • Die Dialoge auf Mobilen Devices entspricht dem Figma
  • Die Trennlinie für den Scrollbaren bereich ist ebenfalls in der Webversion erstellt
  • Das Verhalten ist auf allen Browsern das gleiche (Edge, Firefox, Safari)

Akzeptanzkriterien

  • Dialoge sind angepasst gemäss Anforderungen
  • Tests sind geschrieben
  • UX hat das feature gereviewd
@peggimann peggimann added the request New feature request label Nov 27, 2023
@peggimann peggimann self-assigned this Nov 27, 2023
@lkleisa lkleisa added bug Something isn't working and removed request New feature request labels Dec 14, 2023
@peggimann peggimann added the UX UX is involved label Aug 20, 2024
@kcinay055679
Copy link
Collaborator

In Safari werden die Buttons der Dialoge abgeschnitten.

Image
Um das zu testen kann Browserstack verwendet werden. Browserstack ist nicht nur ein Simulator sondern verwendet echte Geräte welche gemirrored werden. Zusätzlich unterstützt Browserstack auch das Verbinden mit dem eigenen System damit kann sogar auf localhost zugegriffen werden.

@kcinay055679
Copy link
Collaborator

kcinay055679 commented Oct 25, 2024

(Potential) TODO's

  • unify CancelDialog andConfirmDialog
  • Check if all dialog should have height 100% on mobile (Yes they should)
  • Adjust styling of dialogs to Figma
  • create error component (new ticket?)
  • generalize spacing in all components (parent class for the label and .okr-form-input)
  • use .keyResult-attribute-show where appropriate
  • $keyResult-attribute-bg vs $display-element
  • angular material inputs or native?

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

Successfully merging a pull request may close this issue.

4 participants