Bedienung des TimeChoosers
gillesB opened this issue · comments
- Ich fände es gut wenn es immer nur ein TimeChooser geben würde. Atm ist es so dass wenn man auf den Knopf drückt, dass dann ein neuer TimeChooser aufgeht. Wenn man 3 der Dinger hat, dann macht das wenig Sinn.
Dies könnte man mit einem Singleton realisieren, den man nach bedarf sichtbar, unsichtbar macht. - Was ich auch noch gut fände ist, dass wenn man einen Zeitslot auswählt, dass dann ein Aktualisierungs-Event gefeuert wird. Dies passiert jetzt erst beim Schließen des Fensters.
Ich fand dieses Verhalten recht verwirrend und ich hab es nur begriffen weil ich in der Konsole die Nachricht "Received TimeFrames" sah. - Außerdem wäre es gut, wenn die ausgewählten Zeiten gespeichert werden. Macht man einen neuen TimeChooser auf, so muss man wieder alles auswählen. Was korrigieren sehr nervig macht. (Würde man mit dem Singleton aus 1. geschenkt kriegen)
Zu 1.: Also ein Singleton ist vom Design her nicht zu empfehlen. Was noch
rein muss ist, dass beim Öffnen des TimeChoosers dieser immer im
Vordergrund bleibt, dadurch kann man dann auch nicht mehrmals auf den
Button zum Öffnen klicken.
Zu 2.: Ein Aktualisierungsevent macht keinen Sinn, da es einfach keinen
Mehrwert hat. Dies ist ein ganz normales Listener-Verhalten. Sicherlich
könnte man es auch wie beim JFileChooser machen, aber ob das Event jetzt
beim Schließen kommt oder ich danach den Wert abrufe gibt sich nicht viel.
Zu 3.:Hier kommt einfach noch ein Konstruktor dazu bei dem man die alten
TiemFrames mit geben kann.
Singleton würde ich ja jetzt bei GUI allgemein eher vermeiden, da es dem
eigentlichen Grund für seine Verwendung(von mehreren Stellen aus die selbe
Instanz verwenden) wieder spricht.
Wenn ich noch Zeit habe mache ich es noch für Dienstag. Allerdings halte
ich es nicht für schlimm, wenn es bis dahin nicht drin ist.
Am 18. August 2013 15:55 schrieb Gilles Baatz notifications@github.com:
Ich fände es gut wenn es immer nur ein TimeChooser geben würde. Atm
ist es so dass wenn man auf den Knopf drückt, dass dann ein neuer
TimeChooser aufgeht. Wenn man 3 der Dinger hat, dann macht das wenig Sinn.
Dies könnte man mit einem Singleton realisieren, den man nach bedarf
sichtbar, unsichtbar macht.
2.Was ich auch noch gut fände ist, dass wenn man einen Zeitslot
auswählt, dass dann ein Aktualisierungs-Event gefeuert wird. Dies passiert
jetzt erst beim Schließen des Fensters.
Ich fand dieses Verhalten recht verwirrend und ich hab es nur
begriffen weil ich in der Konsole die Nachricht "Received TimeFrames" sah.
3.Außerdem wäre es gut, wenn die ausgewählten Zeiten gespeichert werden.
Macht man einen neuen TimeChooser auf, so muss man wieder alles auswählen.
Was korrigieren sehr nervig macht. (Würde man mit dem Singleton aus 1.
geschenkt kriegen)—
Reply to this email directly or view it on GitHubhttps://github.com//issues/15
.
Ok, finde ich auch alles gut.
On 18.08.2013 18:11, tobiaskalmes wrote:
Zu 1.: Also ein Singleton ist vom Design her nicht zu empfehlen. Was noch
rein muss ist, dass beim Öffnen des TimeChoosers dieser immer im
Vordergrund bleibt, dadurch kann man dann auch nicht mehrmals auf den
Button zum Öffnen klicken.Zu 2.: Ein Aktualisierungsevent macht keinen Sinn, da es einfach keinen
Mehrwert hat. Dies ist ein ganz normales Listener-Verhalten. Sicherlich
könnte man es auch wie beim JFileChooser machen, aber ob das Event jetzt
beim Schließen kommt oder ich danach den Wert abrufe gibt sich nicht
viel.Zu 3.:Hier kommt einfach noch ein Konstruktor dazu bei dem man die alten
TiemFrames mit geben kann.Singleton würde ich ja jetzt bei GUI allgemein eher vermeiden, da es dem
eigentlichen Grund für seine Verwendung(von mehreren Stellen aus die
selbe
Instanz verwenden) wieder spricht.Wenn ich noch Zeit habe mache ich es noch für Dienstag. Allerdings halte
ich es nicht für schlimm, wenn es bis dahin nicht drin ist.Am 18. August 2013 15:55 schrieb Gilles Baatz notifications@github.com:
Ich fände es gut wenn es immer nur ein TimeChooser geben würde. Atm
ist es so dass wenn man auf den Knopf drückt, dass dann ein neuer
TimeChooser aufgeht. Wenn man 3 der Dinger hat, dann macht das wenig
Sinn.
Dies könnte man mit einem Singleton realisieren, den man nach bedarf
sichtbar, unsichtbar macht.
2.Was ich auch noch gut fände ist, dass wenn man einen Zeitslot
auswählt, dass dann ein Aktualisierungs-Event gefeuert wird. Dies
passiert
jetzt erst beim Schließen des Fensters.
Ich fand dieses Verhalten recht verwirrend und ich hab es nur
begriffen weil ich in der Konsole die Nachricht "Received
TimeFrames" sah.
3.Außerdem wäre es gut, wenn die ausgewählten Zeiten gespeichert werden.
Macht man einen neuen TimeChooser auf, so muss man wieder alles
auswählen.
Was korrigieren sehr nervig macht. (Würde man mit dem Singleton aus 1.
geschenkt kriegen)—
Reply to this email directly or view it on
GitHubhttps://github.com//issues/15
.—
Reply to this email directly or view it on GitHub
#15 (comment).
Ist eingebaut. Laden der alten TimeFrames ist drin. Blockieren des ParentFrames ist drin.
Beim Selektieren hab ich mich für die Option "Alles abwählen" und Selektieren eines ganzen Tages entschieden. Damit sollte für Dienstag genug da sein.