On Fri, 17 Feb 2023 12:33:31 GMT, Karthik P K <k...@openjdk.org> wrote:

> When a large number of items were scrolled in the `ChoiceBox`, the scrolled 
> offset was carried forward when the list is replaced with small number of 
> items. Hence the scroll up arrow was displayed with empty popup.
> 
> Changed code to scroll to top before popup display when content height of 
> `ChoiceBox` is smaller than the scrolled offset.
> 
> Added system test to validate the fix.

modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/ContextMenuContent.java
 line 827:

> 825: 
> 826:     boolean isDownArrowVisible() {
> 827:         return upArrow.isVisible();

Just to double-check, shouldn't this be `downArrow.isVisible()`?

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

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

Reply via email to