[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Integrated: 8228363: ContextMenu.show with side=TOP does not work the first time in the presence of 
From:       Robert Lichtenberger <rlichten () openjdk ! java ! net>
Date:       2021-01-27 13:49:43
Message-ID: lYI4bluA6z9rySvN6zzo5H7-mIeBpWx1FM0E31aeMe8=.0ec7d829-87ef-47af-8cb8-96e4dc2aef89 () github ! com
[Download RAW message or body]

On Thu, 21 Jan 2021 06:42:19 GMT, Robert Lichtenberger <rlichten@openjdk.org> wrote:

> By using the anchor location facility of PopupWindows we can avoid miscalculation of the
> menu's height entirely.
> This fix also cleans up some documentation issues.
> This fix introduces tests that check the correct positioning (test_position_)
> test_position_withCSS reproduces the problem that is fixed with this patch.
> The other test_position_ cases serve as "proof" that no regressions are introduces.
> They work before and after the fix is introduced.

This pull request has now been integrated.

Changeset: 6c1a0ca7
Author:    Robert Lichtenberger <rlichten@openjdk.org>
Committer: Kevin Rushforth <kcr@openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/6c1a0ca7
Stats:     192 lines in 3 files changed: 164 ins; 14 del; 14 mod

8228363: ContextMenu.show with side=TOP does not work the first time in the presence of CSS

Reviewed-by: kcr, aghaisas

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

PR: https://git.openjdk.java.net/jfx/pull/383
[prev in list] [next in list] [prev in thread] [next in thread]