Re: RFR: 8090123: Items are no longer visible when collection is changed [v7]

2023-03-06 Thread Karthik P K
On Tue, 7 Mar 2023 00:23:48 GMT, Kevin Rushforth wrote: > Have you run the test this on all platforms? I have run the test in Mac and Windows multiple times. No failures found. > tests/system/src/test/java/test/robot/javafx/scene/ChoiceBoxScrollUpOnCollectionChangeTest.java > line 62: > >>

Re: RFR: 8090123: Items are no longer visible when collection is changed [v8]

2023-03-06 Thread Karthik P K
> 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

Re: RFR: 8090123: Items are no longer visible when collection is changed [v7]

2023-03-06 Thread Kevin Rushforth
On Fri, 3 Mar 2023 07:28:48 GMT, Karthik P K 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

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Kevin Rushforth
On Mon, 6 Mar 2023 21:08:05 GMT, Andy Goryachev wrote: >> We have many tests that access VirtualFlow, so that seems ok. > > would it be possible to update the JBS ticket with a valid code example? That would be very helpful. - PR: https://git.openjdk.org/jfx/pull/1052

Re: RFR: 8299595: Remove terminally deprecated JavaFX GTK 2 library [v10]

2023-03-06 Thread Thiago Milczarek Sayao
On Thu, 2 Mar 2023 17:01:02 GMT, Thiago Milczarek Sayao wrote: >> Simple PR to remove gtk2 library compilation and loading. > > Thiago Milczarek Sayao has updated the pull request incrementally with two > additional commits since the last revision: > > - Rollback + Change Warnings > -

Re: RFR: 8260528: Clean glass-gtk sizing and positioning code [v55]

2023-03-06 Thread Thiago Milczarek Sayao
On Sat, 4 Mar 2023 17:45:05 GMT, Kevin Rushforth wrote: >> Thiago Milczarek Sayao has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Fix bug when window starts maximized and restores to wrong size > >

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Andy Goryachev
On Mon, 6 Mar 2023 21:01:11 GMT, Kevin Rushforth wrote: >> You are right, it is public API. One can't get directly to it, but it is >> possible to subclass the ListView and use the protected `getVirtualFlow()` >> method. > > We have many tests that access VirtualFlow, so that seems ok. would

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Kevin Rushforth
On Mon, 6 Mar 2023 20:54:20 GMT, Andy Goryachev wrote: >> This is the public API of VirtualFlow. I don't know whether it's possible >> with the API of the Control - But the API of the VirtualFlow should be >> stable too, right? > > You are right, it is public API. One can't get directly to

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Kevin Rushforth
On Mon, 6 Mar 2023 16:04:02 GMT, Florian Kirmaier wrote: > Possible fix for VirtualFlow freeze. > > I encountered the problem when experimenting with VirtualFlow. > > Guess @johanvos should take a look. > All tests are still green, so with some luck, this doesn't break anything but > fixes

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Andy Goryachev
On Mon, 6 Mar 2023 20:50:15 GMT, Florian Kirmaier wrote: >> modules/javafx.controls/src/test/java/test/javafx/scene/control/ListViewTest.java >> line 2540: >> >>> 2538: // scroll to 50 and scroll 1 pixel >>> 2539: vf.scrollTo(50); >>> 2540: vf.scrollPixels(1); >> >>

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Florian Kirmaier
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Mon, 6 Mar 2023 16:39:22 GMT, Andy Goryachev

Re: RFR: 8299595: Remove terminally deprecated JavaFX GTK 2 library [v10]

2023-03-06 Thread Kevin Rushforth
On Thu, 2 Mar 2023 23:38:20 GMT, Thiago Milczarek Sayao wrote: > CSR: https://bugs.openjdk.org/browse/JDK-8303493 This is ready to move to "Proposed" now. I left a couple minor comments that will need to be addressed before I formally review it and before it is "Finalized". -

Re: RFR: 8260528: Clean glass-gtk sizing and positioning code [v55]

2023-03-06 Thread Kevin Rushforth
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Mon, 27 Feb 2023 00:13:40 GMT, Thiago Milczarek

Re: RFR: 8303680 Virtual Flow freezes after calling scrollTo and scrollPixels in succession

2023-03-06 Thread Andy Goryachev
On Mon, 6 Mar 2023 16:04:02 GMT, Florian Kirmaier wrote: > Possible fix for VirtualFlow freeze. > > I encountered the problem when experimenting with VirtualFlow. > > Guess @johanvos should take a look. > All tests are still green, so with some luck, this doesn't break anything but > fixes

Re: RFR: 8178368: Right alignment of text fields and alignment of prompt text works incorrectly [v9]

2023-03-06 Thread Andy Goryachev
The message from this sender included one or more files which could not be scanned for virus detection; do not open these files unless you are certain of the sender's intent. -- On Mon, 6 Mar 2023 13:12:05 GMT, Karthik P K

Re: RFR: 8303680 VirtualFlow freeze

2023-03-06 Thread Kevin Rushforth
On Mon, 6 Mar 2023 16:04:02 GMT, Florian Kirmaier wrote: > Possible fix for VirtualFlow freeze. > > I encountered the problem when experimenting with VirtualFlow. > > Guess @johanvos should take a look. > All tests are still green, so with some luck, this doesn't break anything but > fixes

RFR: JDK-8303680 VirtualFlow freeze

2023-03-06 Thread Florian Kirmaier
Possible fix for VirtualFlow freeze. I encountered the problem when experimenting with VirtualFlow. Guess @johanvos should take a look. All tests are still green, so with some luck, this doesn't break anything but fixes some known and unknown bugs. - Commit messages: -

RFR - Virtual Flow Freeze

2023-03-06 Thread Florian Kirmaier
Hi Everyone, Found a bug that can cause the VirtualFlow to no longer scroll, So I've implemented a fix and a test for it. https://github.com/openjdk/jfx/pull/1052 Florian Kirmaier

Integrated: 8303217: Webview loaded webpage is not showing play, volume related buttons for embeded Audio/Video elements

2023-03-06 Thread Jay Bhaskar
On Sun, 26 Feb 2023 16:47:23 GMT, Jay Bhaskar wrote: > Issue: default html5 video element not showing media controls > Details: Media control design has been refactored in the current Webkit Gtk > port. The CSS id > and properties related to media buttons, and slider have been

Re: RFR: 8303217: Webview loaded webpage is not showing play, volume related buttons for embeded Audio/Video elements [v9]

2023-03-06 Thread Hima Bindu Meda
On Mon, 6 Mar 2023 12:55:51 GMT, Jay Bhaskar wrote: >> Issue: default html5 video element not showing media controls >> Details: Media control design has been refactored in the current Webkit Gtk >> port. The CSS id >> and properties related to media buttons, and slider have

Re: RFR: 8178368: Right alignment of text fields and alignment of prompt text works incorrectly [v9]

2023-03-06 Thread Karthik P K
On Mon, 6 Mar 2023 13:12:05 GMT, Karthik P K wrote: >> When Text width was more than TextField width, the logic to update >> `textTranslateX` in `updateCaretOff` method was causing the issue of >> unexpected behavior for Right and Center alignment. >> >> Made changes to update

Re: RFR: 8303217: Webview loaded webpage is not showing play, volume related buttons for embeded Audio/Video elements [v9]

2023-03-06 Thread Kevin Rushforth
On Mon, 6 Mar 2023 12:55:51 GMT, Jay Bhaskar wrote: >> Issue: default html5 video element not showing media controls >> Details: Media control design has been refactored in the current Webkit Gtk >> port. The CSS id >> and properties related to media buttons, and slider have

Re: RFR: 8178368: Right alignment of text fields and alignment of prompt text works incorrectly [v9]

2023-03-06 Thread Karthik P K
> When Text width was more than TextField width, the logic to update > `textTranslateX` in `updateCaretOff` method was causing the issue of > unexpected behavior for Right and Center alignment. > > Made changes to update `textTranslateX` in `updateCaretOff` method only when > text width is

Re: RFR: 8303217: Webview loaded webpage is not showing play, volume related buttons for embeded Audio/Video elements [v9]

2023-03-06 Thread Jay Bhaskar
> Issue: default html5 video element not showing media controls > Details: Media control design has been refactored in the current Webkit Gtk > port. The CSS id > and properties related to media buttons, and slider have been > removed, due to which the > media

Re: RFR: 8178368: Right alignment of text fields and alignment of prompt text works incorrectly [v8]

2023-03-06 Thread Karthik P K
> When Text width was more than TextField width, the logic to update > `textTranslateX` in `updateCaretOff` method was causing the issue of > unexpected behavior for Right and Center alignment. > > Made changes to update `textTranslateX` in `updateCaretOff` method only when > text width is

Re: RFR: 8303217: Webview loaded webpage is not showing play, volume related buttons for embeded Audio/Video elements [v8]

2023-03-06 Thread Jay Bhaskar
> Issue: default html5 video element not showing media controls > Details: Media control design has been refactored in the current Webkit Gtk > port. The CSS id > and properties related to media buttons, and slider have been > removed, due to which the > media