RFR: 8366739: ToolBar: overflow menu with fractional scale (2) [v5]
John Hendrikx
jhendrikx at openjdk.org
Tue Dec 23 15:13:57 UTC 2025
On Tue, 23 Dec 2025 14:39:21 GMT, Cormac Redmond <duke at openjdk.org> wrote:
>> Fix overflow menu triggering due to floating-point precision error.
>>
>> At 1.25 display scaling on Windows, floating-point comparison errors (e.g. 109.60000000000001 > 109.6) cause the overflow menu to appear when the lefthand value is regarded as larger than the righthand value.
>>
>> These should be treated as equal (and therefore not display the overflow menu).
>>
>> This bug can happen in both horizontal and vertical toolbar orientation.
>>
>> The new tests added fail without this fix, and pass with it. An existing test has been re-factored slightly to allow re-use and more flexibility in specifying the scene's root node.
>
> Cormac Redmond has updated the pull request incrementally with one additional commit since the last revision:
>
> 8366739: remove pointless re-assignment of length
Looks good, thank you!
-------------
Marked as reviewed by jhendrikx (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/2016#pullrequestreview-3608395565
More information about the openjfx-dev
mailing list