Integrated: JDK-8187314: All Cells: must show backing data always
Marius Hanl
mhanl at openjdk.org
Thu Nov 2 10:36:15 UTC 2023
On Fri, 4 Aug 2023 18:13:34 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
> Before, the `updateItem` method was called with the new value that was committed via `commitEdit()`.
> This is problematic as developers may setup a commit handler via `setOnEditCommit`, which may reject the edit (or change the value otherwise).
> We therefore do call the `updateItem(-1)` which will also call `updateItem` but with the real underlying value.
>
> Changed and added tests for all 4 cells.
This pull request has now been integrated.
Changeset: ead19536
Author: Marius Hanl <mhanl at openjdk.org>
URL: https://git.openjdk.org/jfx/commit/ead19536437be1b6e3543a3c9c1e0cf67beb2944
Stats: 108 lines in 8 files changed: 94 ins; 2 del; 12 mod
8187314: All Cells: must show backing data always
Reviewed-by: angorya, aghaisas
-------------
PR: https://git.openjdk.org/jfx/pull/1197
More information about the openjfx-dev
mailing list