Re: RFR: 8295725: Update copyright header for files modified in 2022

2022-10-20 Thread Kevin Rushforth
On Thu, 20 Oct 2022 11:31:43 GMT, Ambarish Rapte wrote: > Updating last modified year in copyright header of the files that were > modified during year 2022. Marked as reviewed by kcr (Lead). - PR: https://git.openjdk.org/jfx/pull/923

Re: Platform independent deployment

2022-10-20 Thread Nir Lisker
It doesn't look to me like a big problem, regardless of the size of the project. You just include the modules you want depending on what your application needs and which platforms it targets. In Gradle, it's just 2 lines of code. There is also the JavaFX plugin that might help with this, but it's

Re: Platform independent deployment

2022-10-20 Thread Thomas Reinhardt
Thanks Nir for the links to the other discussions. I got the thing to run with the simple approach of including all artifacts. Probably did miss some before but it's late in the night here :) One thing that still bugs me is that I have to do dependency resolution manually if I want to include

Re: Platform independent deployment

2022-10-20 Thread Nir Lisker
There was a discussion on this some years ago, it started here: https://mail.openjdk.org/pipermail/openjfx-dev/2018-April/021762.html (and continued in https://mail.openjdk.org/pipermail/openjfx-dev/2018-May/021774.html). There might have been another discussion after that, I don't remember.

Re: Platform independent deployment

2022-10-20 Thread Thomas Reinhardt
Interesting. I will repeat my test more carefully. Maybe I am just doing something incredible stupid. But Andy has a good point: why include the java classes at all in the platform-specific jars - shouldn't they just contain the native libraries if all the java code is indeed the same?   

Re: Platform independent deployment

2022-10-20 Thread Andy Goryachev
Good point - are we packaging platform-specific javafx parts incorrectly? -andy From: openjfx-dev on behalf of John Hendrikx Date: Thursday, 2022/10/20 at 13:03 To: openjfx-dev@openjdk.org Subject: Re: Platform independent deployment Correct me if I'm wrong, but all the classes in the

Re: Platform independent deployment

2022-10-20 Thread John Hendrikx
Correct me if I'm wrong, but all the classes in the artifacts for win, linux and mac are actually exactly the same -- this is Java code after all, why would all Java classes for a platform be platform specific?  It doesn't matter which one is packaged.  The platform specific stuff lives in the

Re: RFR: 8290844: Add Skin.install() method [v13]

2022-10-20 Thread Andy Goryachev
> - added Skin.install() > - javadoc changes for Skinnable.setSkin(Skin) Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 26 additional

Re: Platform independent deployment

2022-10-20 Thread Thomas Reinhardt
Hi Andy, This is actually a good suggestion. There are compatibility issues with existing installations (end users could change the commandline/arguments). I will have to check. - Thomas Mit freundlichen Grüßen, Thomas Reinhardt Am 20.10.2022 19:12 schrieb Andy Goryachev :

[jfx17u] RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed

2022-10-20 Thread Johan Vos
clean backport of 8277785: ListView scrollTo jumps to wrong location when CellHeight is…changed Reviewed-by: kcr, fkirmaier, aghaisas - Commit messages: - 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed Changes:

[jfx17u] RFR: 8284665: First selected item of a TreeItem multiple selection gets removed if new items are constantly added to the TreeTableView

2022-10-20 Thread Johan Vos
clean backport of 8284665: First selected item of a TreeItem multiple selection gets removed if new items are constantly added to the TreeTableView Reviewed-by: aghaisas - Commit messages: - 8284665: First selected item of a TreeItem multiple selection gets removed if new items

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

2022-10-20 Thread Thiago Milczarek Sayao
On Thu, 20 Oct 2022 17:04:31 GMT, Thiago Milczarek Sayao wrote: >> This cleans size and positioning code, reducing special cases, code >> complexity and size. >> >> Changes: >> >> - cached extents: 28, 1, 1, 1 are old defaults - modern gnome uses different >> sizes. It does not assume any

Re: Platform independent deployment

2022-10-20 Thread Andy Goryachev
Thomas: if your installer can change the command line it uses to launch java, you could modify the classpath to point to a subdirectory or a set of platform-specific jars. do you think this might work? -andy From: openjfx-dev on behalf of Thomas Reinhardt Date: Thursday, 2022/10/20 at

[jfx17u] RFR: 8291625: DialogPane without header nor headerText nor graphic node adds padding to the left of the content pane

2022-10-20 Thread Johan Vos
clean backport of 8291625: DialogPane without header nor headerText nor graphic node adds padding to the left of the content pane Reviewed-by: aghaisas - Commit messages: - 8291625: DialogPane without header nor headerText nor graphic node adds padding to the left of the content

Re: Platform independent deployment

2022-10-20 Thread Nir Lisker
I use this in a gradle project I have and it works for me. All the OS-specific modules are packaged and I can run the app on all 3 desktop platforms. It seems that you are doing something more special. Maybe someone else has insights. On Thu, Oct 20, 2022 at 8:03 PM Thomas Reinhardt wrote: > >

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

2022-10-20 Thread Thiago Milczarek Sayao
> This cleans size and positioning code, reducing special cases, code > complexity and size. > > Changes: > > - cached extents: 28, 1, 1, 1 are old defaults - modern gnome uses different > sizes. It does not assume any size because it varies - it does cache because > it's unlikely to vary on

Re: Platform independent deployment

2022-10-20 Thread Thomas Reinhardt
Hi Nir, Does not work (I testet it) and it can not work (see below). Also, this is exactly what my naive test was (I did not use maven to copy the artifacts, but the result obviously is the same). It can not work as the implementation classes have the same name and thus the jre can not

Re: RFR: 8222210: JFXPanel popups open at wrong coordinates when using multiple hidpi monitors

2022-10-20 Thread Andy Goryachev
On Thu, 20 Oct 2022 14:37:04 GMT, Johan Vos wrote: > The root problem is actually broader than stated in the JBS issue. This PR > now translates screencoordinates from absolute coordinates into coordinates > that take the platformScale into account. > The whole process is complicated by the

Re: Platform independent deployment

2022-10-20 Thread Nir Lisker
Hi Thomas, Did you try to just specify the platform-specific dependencies in the POM? org.openjfx javafx-graphics 19 win org.openjfx javafx-graphics 19 linux org.openjfx javafx-graphics

Re: RFR: 8295725: Update copyright header for files modified in 2022

2022-10-20 Thread Andy Goryachev
On Thu, 20 Oct 2022 11:31:43 GMT, Ambarish Rapte wrote: > Updating last modified year in copyright header of the files that were > modified during year 2022. Marked as reviewed by angorya (Author). - PR: https://git.openjdk.org/jfx/pull/923

RFR: 8222210: JFXPanel popups open at wrong coordinates when using multiple hidpi monitors

2022-10-20 Thread Johan Vos
The root problem is actually broader than stated in the JBS issue. This PR now translates screencoordinates from absolute coordinates into coordinates that take the platformScale into account. The whole process is complicated by the fact that throughout our code, we use e.g. `x` and `y`

RFR: 8295725: Update copyright header for files modified in 2022

2022-10-20 Thread Ambarish Rapte
Updating last modified year in copyright header of the files that were modified during year 2022. - Commit messages: - copyright header year update Changes: https://git.openjdk.org/jfx/pull/923/files Webrev: https://webrevs.openjdk.org/?repo=jfx=923=00 Issue:

[jfx17u] Integrated: 8295660: Change JavaFX release version to 17.0.6 in jfx17u

2022-10-20 Thread Johan Vos
On Wed, 19 Oct 2022 14:55:40 GMT, Johan Vos wrote: > Update JavaFX 17 release version to 17.0.6 > Fix for JDK-8295660 This pull request has now been integrated. Changeset: 32fd4dec Author:Johan Vos URL: https://git.openjdk.org/jfx17u/commit/32fd4dec86dc8862fd1b4ba03d75a34dbf4d7494

[jfx11u] Integrated: 8295659: Change JavaFX release version to 11.0.18 in jfx11u

2022-10-20 Thread Johan Vos
On Wed, 19 Oct 2022 15:02:42 GMT, Johan Vos wrote: > update javaFX release version to 11.0.18 > Fix for JDK-8295659 This pull request has now been integrated. Changeset: 8e1d7fe0 Author:Johan Vos URL: https://git.openjdk.org/jfx11u/commit/8e1d7fe0f96999db95deb82c4575131338b2bfd4

Platform independent deployment

2022-10-20 Thread Thomas Reinhardt
Hi! Apologizes if this is not the proper list to ask my question. For context: we are using the WebView of JavaFX in our legacy swing based frontend application. For now that is the only component we are using but we might migrate completely at a later point in time. I have an issue with

Integrated: 8278426: ImagePool uses terminally deprecated System.runFinalization method

2022-10-20 Thread Lukasz Kostyra
On Mon, 17 Oct 2022 14:02:52 GMT, Lukasz Kostyra wrote: > This change removes runFinalization calls in ImagePool::pruneCache method. > > Finalizers are deprecated in JDK 18 and will be removed soon. This pull request has now been integrated. Changeset: 91e1a278 Author:Lukasz Kostyra