Thanks for reporting this.
-- Kevin
On 10/10/2022 10:15 AM, Dirk Lemmermann wrote:
Just submitted a bug report related to this issue. The DatePicker could now
commit back to the old / previous value when the user selection of a new date
via the dropdown also triggers the closing of the dialog that is showing the
date picker. In that case the newly added commitValue() method will use the
date expressed by the text inside the editor (TextField), which is actually the
old date.
Dirk
On 7 Dec 2021, at 14:11, Kevin Rushforth <k...@openjdk.java.net> wrote:
On Wed, 24 Nov 2021 09:09:53 GMT, Marius Hanl <mh...@openjdk.org> wrote:
This PR fixes an issue where the `DatePicker` is not committing his text as
value when the focus is lost.
As the ticket also mentions, this is a regression which last worked on JavaFX 8
and got broken by this refactoring:
[JDK-8150946](https://bugs.openjdk.java.net/browse/JDK-8150946)
The fix is to provide the same api to the `DatePicker` which was introduced by
[JDK-8150946](https://bugs.openjdk.java.net/browse/JDK-8150946) for `ComboBox`
and `Spinner`.
Note: While fixing this I found a possible bug which I tracked here:
[JDK-8277756](https://bugs.openjdk.java.net/browse/JDK-8277756)
-> When creating a `DatePicker` with the second constructor (with `LocalDate`
as parameter) two listener won't be added since they are only added at the first
constructor (That's also why I added the focusProperty listener in the second
constructor).
Looks good. I've also "Reviewed" the CSR, so it can be finalized.
-------------
Marked as reviewed by kcr (Lead).
PR: https://git.openjdk.java.net/jfx/pull/679
