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 LGTM ------------- Marked as reviewed by mstrauss (Committer). PR Review: https://git.openjdk.org/jfx/pull/1467#pullrequestreview-2091162499