Integrated: 8252238: TableView: Editable (pseudo-editable) cells should respect the row editability
Marius Hanl
mhanl at openjdk.java.net
Wed Jun 23 09:23:27 UTC 2021
On Sun, 6 Jun 2021 12:44:00 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
> This PR enables Tree- and TableCells to also check the row editability when an edit should happen. With this a Tree- or TableCell is not editable, when the row where the cell is in is not.
>
> While this PR fixes the problem described in the ticket, it does not fix the example.
> This is due the example uses the **CheckBoxTableCell**, which is a ready-to-use subclass of **TableCell**.
>
> While looking into this, I found out that multiple sub implementations still have this issue, but the fix is not always the same, e.g. CheckBoxTableCell should disable the CheckBox (in **updateItem**), while the ChoiceBoxTableCell should check the row editability in the **startEdit** method (like this PR does).
>
> I created a follow-up issues for fixing all the sub Tree- and TableCell implementation which do not count the row editability in:
> [JDK-8268295](https://bugs.openjdk.java.net/browse/JDK-8268295)
This pull request has now been integrated.
Changeset: 45786ac4
Author: Marius Hanl <mhanl at openjdk.org>
Committer: Ajit Ghaisas <aghaisas at openjdk.org>
URL: https://git.openjdk.java.net/jfx/commit/45786ac4f6c3abda45b162b27d158e2dee9e9c14
Stats: 169 lines in 4 files changed: 159 ins; 0 del; 10 mod
8252238: TableView: Editable (pseudo-editable) cells should respect the row editability
Reviewed-by: fastegal, aghaisas
-------------
PR: https://git.openjdk.java.net/jfx/pull/529
More information about the openjfx-dev
mailing list