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

proof of concept - range validation #43

Merged
merged 1 commit into from Jul 25, 2018

Conversation

nittka
Copy link
Contributor

@nittka nittka commented Jul 25, 2018

In diesem PR geht es um die grundsätzliche Frage, ob für Zeiträume eine Plausibilitätsprüfung gemacht werden sollte. Ist ein Zeitraum invalid (bis liegt vor von), wird es keine Treffer geben. Es ist aber nicht immer klar, ob dem Nutzer der Fehler auffällt. Ich halte diese Prüfung und eine entsprechende Rückmeldung an den Nutzer für sinnvoll.

In dieser Minimalumsetzung - wirklich nur proof of concept - wird eine Fehlernachricht eingeblendet (die aber natürlich nicht sofort verschwindet, wenn der Fehler behoben ist). Eine andere Möglichkeit wäre, den bis-Wert automatisch auf den von-Wert zu korrigieren oder das bis-Feld einzufärben (es sah aber ziemlich unschön aus, den background rot zu machen).

@willuhn
Copy link
Owner

willuhn commented Jul 25, 2018

Danke für den PR.

@willuhn willuhn merged commit 8f2c177 into willuhn:master Jul 25, 2018
willuhn added a commit that referenced this pull request Jul 25, 2018
…rwähnten Feld-Bezeichner zu den angezeigten Feld-Bezeichnern passen.
@nittka
Copy link
Contributor Author

nittka commented Jul 25, 2018

Bitte den Text nochmal prüfen... Von- und Bis-Datum sind vertauscht.

Du siehst kein Problem darin, dass die Nachricht nicht sofort verschwindet, wenn der Fehler behoben ist? Man gewöhnt sich vielleicht dran, aber ein nicht (mehr) zutreffender Hinweis verunsichert schon...

@nittka nittka deleted the poc_rangevalidation branch July 25, 2018 07:57
willuhn added a commit that referenced this pull request Jul 25, 2018
@willuhn
Copy link
Owner

willuhn commented Jul 25, 2018

Bitte den Text nochmal prüfen... Von- und Bis-Datum sind vertauscht.

Hopple. Danke für den Hinweis.

Du siehst kein Problem darin, dass die Nachricht nicht sofort verschwindet,
wenn der Fehler behoben ist?

Habs noch ergaenzt. Allerdings ist das nicht ganz sauber. Denn eigentlich duerfte der Code ja nur die eigenen Hinweise ausblenden. Sprich: Falls gerade eine andere Fehlermeldung angezeigt wird, man dann in eines der Datumsfelder klickt, dann wird auch diese ausgeblendet.

Man muesste den Code in Jameica also eigentlich so ergaenzen, dass die Notification eine Art Identifier hat und man beim Loeschen diesen mit angeben kann, sodass sie nur dann ausgeblendet wird, wenn es auch wirklich die zu loeschende Notification ist. Das gibt die Jameica-API derzeit aber nicht her. Und eine Erweiterung waere nicht abwaertskompatibel. Und wegen einer so kleinen Aenderung wuerde ich ungern die Abwaertskompatibilitaet brechen.

@nittka
Copy link
Contributor Author

nittka commented Jul 25, 2018

Ich wollte kein großes Thema draus machen. Wie gesagt, war der Ansatz mit der Notification auch nur als Beispiel gedacht. Eine Feldvalidierung, bei der der Fehler direkt beim Feld angezeigt wird, wäre eigenlich besser geeignet. (Sowas ähnliches wie die Pflichtfeldeinfärbung, wenn der Wert fehlt...)

Wenn Du mit dieser Lösung leben kannst, habe ich gar kein Problem.

@willuhn
Copy link
Owner

willuhn commented Jul 25, 2018

Eine Feldvalidierung, bei der der Fehler direkt beim Feld angezeigt wird,
wäre eigenlich besser geeignet. (Sowas ähnliches wie die Pflichtfeldeinfärbung,
wenn der Wert fehlt...)

Ja, sowas würde mir auch besser gefallen. So in Form von Tooltips, die wie bei Formularen auf Webseiten direkt neben den Eingabefeldern angezeigt werden. Leider gibts in SWT hierfür aber keine fertigen Komponenten. Das müsste man komplett selbst zeichnen.

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