"active" checkbox is not working well in the All features tab

[javier-godoy]

Describe the bug

Moving comment by @mlopezFC from #37 (comment)

I see that the "active" checkbox is not working well in the All features tab. If I put all the items as "active" some of them still work as inactive and cause the flickering.

The "active" column has been changed from read-only to writeable:

https://github.com/FlowingCode/GridHelpers/blame/3a2a3dae93b305e4a1f504878eba84e7f1c13b9e/src/test/java/com/flowingcode/vaadin/addons/gridhelpers/AllFeaturesDemo.java#L78-L79

https://github.com/FlowingCode/GridHelpers/blame/3abdb2eddf93bbe5de835995bbba90ace8286716/src/test/java/com/flowingcode/vaadin/addons/gridhelpers/AllFeaturesDemo.java#L82-L84

Moreover, it's not clear how to reflect the changes of a CheckBoxColumn

Expected behavior

No response

Minimal reproducible example

No response

Add-on Version

1.4.0-SNAPSHOT

Vaadin Version

24

Additional information

No response

[javier-godoy]

CheckBoxColumn has a checkboxClickListener

GridHelpers/src/main/java/com/flowingcode/vaadin/addons/gridhelpers/CheckboxColumn.java

Line 93 in 190f038

private SerializableBiConsumer<T, Boolean> checkboxClickListener;

The listener, however, isn't mentioned in the Javadoc.

[mlopezFC]

So if I understand correctly there are two possible solutions:

• Change the active column back to be readonly
• Use the listener to update the internal status so it will reflect the changes when clicking on "Disallow selection of 'inactive' records"
  Moving to todo so the developer that will fix this can decide which one of these solutions can implement