RFR: 8185887: TableRowSkinBase fails to correctly virtualize cells in horizontal direction [v4]

[Marius Hanl]

On Thu, 16 Jan 2025 15:47:04 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> That is right, but the `pulse` call will call `layout`, which will call `layoutChildren` (when needed). I could also just call `layout` directly, but want to keep it the same way as the actual Application would behave.
>
> Relying on two pulses may cause flicker (I had a similar issue with the RichTextArea).
> 
> The solution is to re-compute the layout in layoutChildren() when needed (i.e. when scroll bar status changes).

Good to know, we should document your solution in more detail (and with the concrete fix -> code snippet from the RTA) together with the HSB bug as a new ticket probably? Then we may already have a potential solution with code, and of course a ticket that tracks the bug, if we do not have already (I think I saw a similar bug already, but not 100% sure).

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1644#discussion_r1919332413