search

JFXtras / jfxtras

A supporting library for JavaFX, containing helper classes, extended layouts, controls and other interesting widgets.
http://jfxtras.org
Other
599 stars 123 forks source link

added appointmentChangedCallback #51

Closed daviddbal closed 8 years ago

daviddbal commented 8 years ago

When an appointment is changed by Agenda (e.g. drag-n-drop to new time) change listeners will not fire. To enable the client to process those changes this callback can be used. Additionally, for a repeatable appointment, this can be used to prompt the user if they want the change to occur to one, this-and-future or all events in series.

No error in string this time. I also shut off auto-import-reorganize to minimize changes.

tbee commented 8 years ago

You did not implement the event on all places where appointments are modified. I'm willing to merge this PR, but only if I have the time to right away add the other places as well. I cannot let a partial implementation go out in public.

daviddbal commented 8 years ago

I can put it in the other places. Which place(s) did I miss? You can describe the action and I should be able to find the location in the code.

daviddbal commented 8 years ago

I added a oneAppointmentSelected callback as you discussed in a previous email.