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