Integrated: 8328577: Toolbar's overflow button overlaps the items

eduardsdv duke at openjdk.org
Tue Apr 23 11:49:35 UTC 2024


On Thu, 28 Mar 2024 11:31:14 GMT, eduardsdv <duke at openjdk.org> wrote:

> This change fixes the calculation of which nodes go to the toolbar and which go to the overflow menu.
> It is now determined before the nodes are removed from the scene graph.
> This is important because the values returned by ``Node.prefWidth(..)``/``Node.prefHeight(..)`` may depend on whether the node is added to the scene graph.
> 
> Furthermore I corrected the ``hasOveflow`` value passed to the ``organizeOverflow(double, boolean)`` in ``correctOverflow(double)``.

This pull request has now been integrated.

Changeset: 3333740f
Author:    Eduard Sedov <eduard.sedov at zeiss.com>
Committer: Karthik P K <kpk at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/3333740f1faa5459027f5d078241ce0dc3a9f9cf
Stats:     275 lines in 2 files changed: 182 ins; 62 del; 31 mod

8328577: Toolbar's overflow button overlaps the items

Reviewed-by: angorya, kpk

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

PR: https://git.openjdk.org/jfx/pull/1434


More information about the openjfx-dev mailing list