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

Robert Lichtenberger rlichten at openjdk.java.net
Mon Jan 25 11:35:44 UTC 2021


On Fri, 22 Jan 2021 19:01:13 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Robert Lichtenberger has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8228363: ContextMenu.show with side=TOP does not work the first time in the presence of CSS
>>   
>>   Corrections as per Kevin Rushforth's comments.
>>   Also added two more test cases that test right-to-left node orientation.
>>   Fixed the implementation and the API documentation.
>
> modules/javafx.controls/src/main/java/javafx/scene/control/ContextMenu.java line 302:
> 
>> 300: 
>> 301: 
>> 302: 
> 
> Minor: there should only be one blank line here.

Agreed. Additional lines removed.

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

PR: https://git.openjdk.java.net/jfx/pull/383


More information about the openjfx-dev mailing list