RFR: 8328577: Toolbar's overflow button overlaps the items [v6]

eduardsdv duke at openjdk.org
Mon Apr 15 15:41:19 UTC 2024


> 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)``.

eduardsdv has updated the pull request incrementally with one additional commit since the last revision:

  JDK-8328577: Refactor and fix binding of style related properties

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1434/files
  - new: https://git.openjdk.org/jfx/pull/1434/files/e63b0b9b..9b159e3b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1434&range=05
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1434&range=04-05

  Stats: 6 lines in 1 file changed: 1 ins; 2 del; 3 mod
  Patch: https://git.openjdk.org/jfx/pull/1434.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1434/head:pull/1434

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


More information about the openjfx-dev mailing list