Re: RFR: JDK-8285893: Hiding dialog and showing new one causes dialog to be frozen [v3]

2024-03-01 Thread Martin Fox
On Fri, 1 Mar 2024 18:44:17 GMT, Marius Hanl wrote: >> This PR fixes the dialog freeze problem once and for all. >> >> This one is a bit tricky to understand, here is how it works: >> This bug happens on every platform, although the implementation of nested >> event loops differs on every plat

Re: Proposal: RichTextArea Control (Incubator)

2024-03-01 Thread Kevin Rushforth
All, Thank you to everyone who has provided feedback on the RichText proposal so far. If I could characterize the response, there are some developers who want at least a basic rich text control available as part of the JavaFX core and other developers who don't think that such a control shoul

Re: Proposal: JavaFX Incubator Modules

2024-03-01 Thread Kevin Rushforth
I updated the Incubator Modules proposal to address Phil's feedback, along with updating the example incubator module: https://github.com/kevinrushforth/jfx/blob/javafx.incubator/INCUBATOR-MODULES.md https://github.com/openjdk/jfx/pull/1375 -- Kevin On 2/22/2024 2:32 PM, Kevin Rushforth wrote

Re: RFR: JDK-8285893: Hiding dialog and showing new one causes dialog to be frozen [v3]

2024-03-01 Thread Marius Hanl
On Fri, 1 Mar 2024 21:33:16 GMT, Martin Fox wrote: > The output I would expect would be: > > ``` > enter1 > exit1 > enter2 > exit2 > 456 > 123 > ``` You are right, this sounds correct. > By the way, when I grab the code in the original bug report and run it agains > this PR the innermost moda

Re: RFR: JDK-8285893: Hiding dialog and showing new one causes dialog to be frozen [v3]

2024-03-01 Thread Martin Fox
On Fri, 1 Mar 2024 18:44:17 GMT, Marius Hanl wrote: >> This PR fixes the dialog freeze problem once and for all. >> >> This one is a bit tricky to understand, here is how it works: >> This bug happens on every platform, although the implementation of nested >> event loops differs on every plat

Re: RFR: JDK-8285893: Hiding dialog and showing new one causes dialog to be frozen [v3]

2024-03-01 Thread Marius Hanl
On Fri, 1 Mar 2024 18:44:17 GMT, Marius Hanl wrote: >> This PR fixes the dialog freeze problem once and for all. >> >> This one is a bit tricky to understand, here is how it works: >> This bug happens on every platform, although the implementation of nested >> event loops differs on every plat

Re: RFR: JDK-8285893: Hiding dialog and showing new one causes dialog to be frozen [v3]

2024-03-01 Thread Marius Hanl
> This PR fixes the dialog freeze problem once and for all. > > This one is a bit tricky to understand, here is how it works: > This bug happens on every platform, although the implementation of nested > event loops differs on every platform. > E.g. on Linux we use `gtk_main` and `gtk_main_quit`

Integrated: 8325073: javadoc warnings: missing @param tags and other issues

2024-03-01 Thread Andy Goryachev
On Wed, 28 Feb 2024 21:48:38 GMT, Andy Goryachev wrote: > This change brings the number of javadoc warnings back to 91 (to be fixed in > [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) > > - adds missing information in `@param` tags > - adds `@SuppressWarnings("doclint:missing")` t

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v3]

2024-03-01 Thread Andy Goryachev
On Fri, 1 Mar 2024 15:53:21 GMT, Andy Goryachev wrote: >> This change brings the number of javadoc warnings back to 91 (to be fixed in >> [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) >> >> - adds missing information in `@param` tags >> - adds `@SuppressWarnings("doclint:missing"

Re: RFR: 8270996: javadoc: missing comments in serialized classes [v3]

2024-03-01 Thread Kevin Rushforth
On Fri, 1 Mar 2024 16:19:14 GMT, Andy Goryachev wrote: >> This PR is addressing doclint warnings, not javac lint warnings. javac >> warnings can be addressed in a follow-up bug. > > FYI: after merging this and #1384 together all I see in the build log are 4 > native warnings (on macOS): > > >

Re: running systemtests in isolated java processes

2024-03-01 Thread Johan Vos
On Fri, Mar 1, 2024 at 4:04 PM Kevin Rushforth wrote: > Hi Johan, > > FOr the systemTests project, the gradle test runner does run each test > class in its own VM, as a result of the "forkEvery = 1" setting. As > noted, gradle launches tests using a gradle test runner class as the > main method,

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v3]

2024-03-01 Thread Ambarish Rapte
On Fri, 1 Mar 2024 15:53:21 GMT, Andy Goryachev wrote: >> This change brings the number of javadoc warnings back to 91 (to be fixed in >> [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) >> >> - adds missing information in `@param` tags >> - adds `@SuppressWarnings("doclint:missing"

Re: RFR: 8270996: javadoc: missing comments in serialized classes [v3]

2024-03-01 Thread Andy Goryachev
On Fri, 1 Mar 2024 13:05:23 GMT, Kevin Rushforth wrote: >> modules/javafx.swing/src/main/java/javafx/embed/swing/JFXPanel.java line 142: >> >>> 140: * future Swing or JavaFX releases. The current serialization support >>> is >>> 141: * appropriate for short term storage or RMI between applica

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v3]

2024-03-01 Thread Kevin Rushforth
On Fri, 1 Mar 2024 15:53:21 GMT, Andy Goryachev wrote: >> This change brings the number of javadoc warnings back to 91 (to be fixed in >> [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) >> >> - adds missing information in `@param` tags >> - adds `@SuppressWarnings("doclint:missing"

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v2]

2024-03-01 Thread Andy Goryachev
On Fri, 1 Mar 2024 03:40:57 GMT, Ambarish Rapte wrote: > there is no empty line between the description and `@param` it makes no difference in the generated javadoc output. I've made the changes anyway, for the developers' sake. - PR Comment: https://git.openjdk.org/jfx/pull/1384#i

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v3]

2024-03-01 Thread Andy Goryachev
> This change brings the number of javadoc warnings back to 91 (to be fixed in > [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) > > - adds missing information in `@param` tags > - adds `@SuppressWarnings("doclint:missing")` to `Skinnable` to silence the > warning due to [JDK-832507

Re: RFR: 8325073: javadoc warnings: missing @param tags and other issues [v2]

2024-03-01 Thread Kevin Rushforth
On Fri, 1 Mar 2024 00:22:05 GMT, Andy Goryachev wrote: >> This change brings the number of javadoc warnings back to 91 (to be fixed in >> [JDK-8270996](https://bugs.openjdk.org/browse/JDK-8270996)) >> >> - adds missing information in `@param` tags >> - adds `@SuppressWarnings("doclint:missing"

Re: running systemtests in isolated java processes

2024-03-01 Thread Kevin Rushforth
Hi Johan, FOr the systemTests project, the gradle test runner does run each test class in its own VM, as a result of the "forkEvery = 1" setting. As noted, gradle launches tests using a gradle test runner class as the main method, but there is still a 1 VM to 1 test class relationship. We ha

Re: RFR: 8319779: SystemMenu: memory leak due to listener never being removed [v10]

2024-03-01 Thread Johan Vos
On Tue, 27 Feb 2024 09:30:14 GMT, Johan Vos wrote: >> A listener was added but never removed. >> This patch removes the listener when the menu it links to is cleared. Fix >> for https://bugs.openjdk.org/browse/JDK-8319779 > > Johan Vos has updated the pull request with a new target base due to a

running systemtests in isolated java processes

2024-03-01 Thread Johan Vos
Hi, We currently run systemtests using a gradle command, but that doesn't start a clean Java process for the individual tests (which would be very convenient as that allows us to inspect the process with VisualVM). We know the behavior of a JavaFX application can be different depending if 1. the m

Re: RFR: 8270996: javadoc: missing comments in serialized classes [v3]

2024-03-01 Thread Kevin Rushforth
On Fri, 1 Mar 2024 03:49:20 GMT, Prasanta Sadhukhan wrote: >> Andy Goryachev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comments > > modules/javafx.swing/src/main/java/javafx/embed/swing/JFXPanel.java line 142: > >> 140: *

[jfx17u] Integrated: 8322703: Intermittent crash in WebView in a JFXPanel from IME calls on macOS

2024-03-01 Thread Jose Pereda
On Thu, 29 Feb 2024 15:48:30 GMT, Jose Pereda wrote: > Almost clean backport of 8322703: Intermittent crash in WebView in a JFXPanel > from IME calls on macOS > > Reviewed-by: jbhaskar, arapte > > Very minor conflicts after cherry picking the commit: > > // InputMethodRequests implementa

Validation Support

2024-03-01 Thread Dirk Lemmermann
Hi everyone, I updated the validation framework ValidatorFX today in our project to the latest release and I really like it a lot. It is a small compact API and works with any observable as opposed to the validation support provided by ControlsFX. Using it made me wonder whether it would make

Re: [jfx17u] RFR: 8322703: Intermittent crash in WebView in a JFXPanel from IME calls on macOS

2024-03-01 Thread Johan Vos
On Thu, 29 Feb 2024 15:48:30 GMT, Jose Pereda wrote: > Almost clean backport of 8322703: Intermittent crash in WebView in a JFXPanel > from IME calls on macOS > > Reviewed-by: jbhaskar, arapte > > Very minor conflicts after cherry picking the commit: > > // InputMethodRequests implementa