On Fri, 31 May 2024 15:17:19 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> The root cause is that the skin used two fields to store one entity 
>> (`focusedMenu` and `focusedMenuIndex`), causing mismatch when invisible 
>> menu(s) are present.
>> 
>> The fix involves using a single index variable.
>> 
>> Also wanted to note that in theory, `openMenu` and `openMenuButton` can also 
>> be replaced with a single boolean, but I decided not to change these in 
>> order to keep the amount of diffs to a minimum.
>
> Andy Goryachev has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   review comments

@aghaisas or @arapte could you please take a look at this PR?

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

PR Comment: https://git.openjdk.org/jfx/pull/1467#issuecomment-2142598529

Reply via email to