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

calendar: dialog - refactoring the warning messages #533

Closed
andreaschu opened this issue Aug 9, 2022 · 6 comments
Closed

calendar: dialog - refactoring the warning messages #533

andreaschu opened this issue Aug 9, 2022 · 6 comments

Comments

@andreaschu
Copy link
Contributor

We refactoring the warning messages with bootstrap warning modal (https://jsfiddle.net/54y82xn9/) because of iOS.

Originally posted by @andreaschu in #472 (comment)

@andreaschu andreaschu changed the title We refactoring the warning messages with bootstrap warning modal (https://jsfiddle.net/54y82xn9/) because of iOS. calendar: dialog - refactoring the warning messages Aug 9, 2022
@andreaschu andreaschu added this to the Sprint 48: milestone Aug 9, 2022
@christian-fr
Copy link
Collaborator

Überarbeitung des Modal bis nächste Woche

  • graphisches Kalendarium aktuelle Logik ausgebaut;
  • neu eingebaut:
  • es wird validiert, ob es einen Typ gibt -> falls es keinen Typ gibt,
    • wird ein Warnhinweis angezeigt,
    • das Label vom „Speichern“-Button wird zu „Erneut prüfen“
    • und die Episode kann nicht gespeichert werden;
  • wird ein Typ gesetzt und auf „Erneut prüfen“ geklickt, dann wird nach erfolgreicher
    Validierung direkt gespeichert

@vdick
Copy link

vdick commented Aug 23, 2022

bitte in Test_Modul testen(ist bereits ausgerollt)

@andreaschu
Copy link
Contributor Author

Works as in #533 (comment).
I don't close the issue because some actions are still open here.

@andreaschu
Copy link
Contributor Author

andreaschu commented Aug 30, 2022

still missing:

  • type change (softforce)
  • apply to table calendar
  • validation of dates (softforce)
  • modal only closeable via button

@vdick
Copy link

vdick commented Sep 9, 2022

bitte in Test_Modul testen (bereits neu ausgerollt), bei allen anderen erst nach neuem Build&Rollout

@andreaschu
Copy link
Contributor Author

only layout is not nice yet. (at least in rolled out version)

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

No branches or pull requests

3 participants