RFR: 8191995: Regression: DatePicker must commit on focusLost

Jeanette Winzenburg fastegal at openjdk.java.net
Thu Nov 25 12:52:03 UTC 2021


On Wed, 24 Nov 2021 09:09:53 GMT, Marius Hanl <mhanl at openjdk.org> wrote:

> This PR fixes an issue where the `DatePicker` is not committing his 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 the 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).

will review thoroughly soon - for now just a comment to get rid of that annoying yellow bar at the top ;)

On first skim, it looks straightforward - adding the api same as for combo/spinner. I think you might get going and flesh out the csr.

-------------

PR: https://git.openjdk.java.net/jfx/pull/679


More information about the openjfx-dev mailing list