Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v6]

2024-03-08 Thread John Hendrikx
> Improves performance of selector matching in the CSS subsystem. This is done > by using custom set implementation which are highly optimized for the most > common cases where the number of selectors is small (most commonly 1 or 2). > It also should be more memory efficient for medium sized and

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread John Hendrikx
On Sat, 9 Mar 2024 00:04:04 GMT, Andy Goryachev wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Optimize performance of OpenAddressed Set just in case it is ever used > > modules/javafx.graphics/src/test/java/test

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread John Hendrikx
On Fri, 8 Mar 2024 23:46:16 GMT, Andy Goryachev wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Optimize performance of OpenAddressed Set just in case it is ever used > > modules/javafx.graphics/src/main/java/com/

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread John Hendrikx
On Fri, 8 Mar 2024 23:26:05 GMT, Andy Goryachev wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Optimize performance of OpenAddressed Set just in case it is ever used > > modules/javafx.graphics/src/main/java/com/

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread John Hendrikx
On Fri, 8 Mar 2024 23:17:28 GMT, Andy Goryachev wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Optimize performance of OpenAddressed Set just in case it is ever used > > modules/javafx.graphics/src/main/java/com/

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread John Hendrikx
On Fri, 8 Mar 2024 23:14:12 GMT, Andy Goryachev wrote: >> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Optimize performance of OpenAddressed Set just in case it is ever used > > modules/javafx.graphics/src/main/java/com/

Re: RFR: JDK-8290310: ChangeListener events are incorrect or misleading when a nested change occurs [v11]

2024-03-08 Thread John Hendrikx
On Fri, 8 Mar 2024 18:08:15 GMT, Marius Hanl wrote: > Tested with a big application, did not find any regression. All listeners > still work as expected, tested especially a lot of 'if something is selected, > this button should be enabled/disabled' and the like. I did not checked the > code y

Re: Aw: Re: Validation Support

2024-03-08 Thread John Hendrikx
I think there is definitely room for improvement here, but I think most controls can currently be build externally, if you are willing to do all the work.  Your control may suffer however when you need to integrate with event, focus handling or keyboard navigation.  Skins and Behaviors are not

Re: RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed [v2]

2024-03-08 Thread John Hendrikx
On Fri, 8 Mar 2024 16:10:26 GMT, Marius Hanl wrote: >> This PR fixes a long standing issue where the `Tooltip` will always wait one >> second until it appears the very first time, even if the >> `-fx-show-delay` was set to another value. >> >> The culprit is, that the `cssForced` flag is not i

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread Andy Goryachev
On Sun, 25 Feb 2024 22:28:12 GMT, John Hendrikx wrote: >> Improves performance of selector matching in the CSS subsystem. This is done >> by using custom set implementation which are highly optimized for the most >> common cases where the number of selectors is small (most commonly 1 or 2). >>

Re: RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed [v2]

2024-03-08 Thread Andy Goryachev
On Fri, 8 Mar 2024 16:10:26 GMT, Marius Hanl wrote: >> This PR fixes a long standing issue where the `Tooltip` will always wait one >> second until it appears the very first time, even if the >> `-fx-show-delay` was set to another value. >> >> The culprit is, that the `cssForced` flag is not i

Re: RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed [v2]

2024-03-08 Thread Andy Goryachev
On Fri, 8 Mar 2024 16:10:26 GMT, Marius Hanl wrote: >> This PR fixes a long standing issue where the `Tooltip` will always wait one >> second until it appears the very first time, even if the >> `-fx-show-delay` was set to another value. >> >> The culprit is, that the `cssForced` flag is not i

Re: RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed [v2]

2024-03-08 Thread Andy Goryachev
On Fri, 8 Mar 2024 16:10:26 GMT, Marius Hanl wrote: >> This PR fixes a long standing issue where the `Tooltip` will always wait one >> second until it appears the very first time, even if the >> `-fx-show-delay` was set to another value. >> >> The culprit is, that the `cssForced` flag is not i

Re: RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed [v2]

2024-03-08 Thread Andy Goryachev
On Fri, 8 Mar 2024 16:10:26 GMT, Marius Hanl wrote: >> This PR fixes a long standing issue where the `Tooltip` will always wait one >> second until it appears the very first time, even if the >> `-fx-show-delay` was set to another value. >> >> The culprit is, that the `cssForced` flag is not i

Re: [External] : Re: Q: Labeled.textTruncated Property (RFE)

2024-03-08 Thread Andy Goryachev
Daniel: Thank you, both the PR and the code have been updated. This change provides an illustration of the problem we have with JavaFX - it is nearly impossible to talk about custom skin development when most of the essential machinery is hidden behind accessors and non-public classes. In this

Re: RFR: 8092102: Labeled: truncated property [v4]

2024-03-08 Thread Andy Goryachev
> Adds **Labeled.textTruncated** property which indicates when the text is > visually truncated (and the ellipsis string is inserted) in order to fit the > available width. > > The new property reacts to changes in the following properties: > - ellipsisString > - font > - height > - text > - wid

Re: Re: Validation Support

2024-03-08 Thread Andy Goryachev
* Offtopic: Maybe we should collect the things which should be public in our opinion at one point to (better) get this started as well? My sentiment exactly! Marius, would you like to create a new thread? Thanks! -andy From: openjfx-dev on behalf of Marius Hanl Date: Thursday, March 7,

Withdrawn: JDK-8218745: TableView: visual glitch at borders on horizontal scrolling

2024-03-08 Thread duke
On Wed, 10 Jan 2024 19:21:16 GMT, Marius Hanl wrote: > This PR fixes the border glitch/gap as explained in both linked tickets. > > I noted that the `ScrollPane` control does not suffer from this problem, > although the code is mostly the same as in `VirtualFlow`. The `ScrollPane` > snaps the

Re: RFR: JDK-8290310: ChangeListener events are incorrect or misleading when a nested change occurs [v11]

2024-03-08 Thread Marius Hanl
On Fri, 9 Jun 2023 12: 00: 06 GMT, John Hendrikx wrote: >> This provides and uses a new implementation of `ExpressionHelper`, called `ListenerManager` with improved semantics. >> >> # Behavior >> ZjQcmQRYFpfptBannerStart This Message Is From an External Sender

Re: RFR: JDK-8322964 Optimize performance of CSS selector matching [v5]

2024-03-08 Thread Marius Hanl
On Sun, 25 Feb 2024 22: 28: 12 GMT, John Hendrikx wrote: >> Improves performance of selector matching in the CSS subsystem. This is done by using custom set implementation which are highly optimized for the ZjQcmQRYFpfptBannerStart This Message Is From an External Send

Withdrawn: 8314968: Public InputMap

2024-03-08 Thread Andy Goryachev
On Tue, 3 Oct 2023 19: 10: 14 GMT, Andy Goryachev wrote: > Summary: > - public InputMap, KeyBinding, BehaviorBase in javafx. scene. control. behavior package > - InputMap allows for adding key mappings on control ZjQcmQRYFpfptBannerStart This Message Is From an Externa

Re: RFR: JDK-8326619: Stage.sizeToScene() on maximized/fullscreen Stage breaks the Window

2024-03-08 Thread Marius Hanl
On Wed, 28 Feb 2024 23:43:40 GMT, Kevin Rushforth wrote: > This seems like a reasonable fix and spec change. Have you tested the case of > calling sizeToScene before setting full-screen or maximzed? Since the pending > flag will still be set in that case, I want to make sure that case is tested

Re: RFR: JDK-8326619: Stage.sizeToScene() on maximized/fullscreen Stage breaks the Window [v2]

2024-03-08 Thread Marius Hanl
> This PR fixes the problem that maximizing/fullscreen a `Stage` or `Dialog` is > broken when `sizeToScene()` was called before or after. > > The approach here is to ignore the `sizeToScene()` request when the `Stage` > is maximized or set to fullscreen. > Otherwise the Window Manager of the OS

Re: RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed [v2]

2024-03-08 Thread Marius Hanl
> This PR fixes a long standing issue where the `Tooltip` will always wait one > second until it appears the very first time, even if the > `-fx-show-delay` was set to another value. > > The culprit is, that the `cssForced` flag is not inside `Tooltip`, but inside > the `TooltipBehaviour`. So t

RFR: JDK-8296387: [Tooltip, CSS] -fx-show-delay is only applied to the first tooltip that is shown before it is displayed

2024-03-08 Thread Marius Hanl
This PR fixes a long standing issue where the `Tooltip` will always wait one second until it appears the very first time, even if the `-fx-show-delay` was set to another value. The culprit is, that the `cssForced` flag is not inside `Tooltip`, but inside the `TooltipBehvaiour`. So the very firs

[jfx17u] Integrated: 8326989: Text selection issues on WebView after WebKit 617.1

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 10:18:21 GMT, Johan Vos wrote: > Hi all, > > This pull request contains a backport of commit 64af9875 from the openjdk/jfx > repository. > > The commit being backported was authored by Hima Bindu Meda on 7 Mar 2024 and > was reviewed by Kevin Rushforth and Jose Pereda. > >

[jfx17u] RFR: 8326989: Text selection issues on WebView after WebKit 617.1

2024-03-08 Thread Johan Vos
Hi all, This pull request contains a backport of commit 64af9875 from the openjdk/jfx repository. The commit being backported was authored by Hima Bindu Meda on 7 Mar 2024 and was reviewed by Kevin Rushforth and Jose Pereda. Thanks! - Commit messages: - Backport 64af987576214fc5

[jfx21u] Integrated: 8326989: Text selection issues on WebView after WebKit 617.1

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 10:13:27 GMT, Johan Vos wrote: > Hi all, > > This pull request contains a backport of commit 64af9875 from the openjdk/jfx > repository. > > The commit being backported was authored by Hima Bindu Meda on 7 Mar 2024 and > was reviewed by Kevin Rushforth and Jose Pereda. > >

[jfx21u] Integrated: 8326989: Text selection issues on WebView after WebKit 617.1

2024-03-08 Thread Johan Vos
Hi all, This pull request contains a backport of commit 64af9875 from the openjdk/jfx repository. The commit being backported was authored by Hima Bindu Meda on 7 Mar 2024 and was reviewed by Kevin Rushforth and Jose Pereda. Thanks! - Commit messages: - Backport 64af987576214fc5

[jfx17u] Integrated: 8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 10:09:53 GMT, Johan Vos wrote: > 8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5 This pull request has now been integrated. Changeset: eb922637 Author:Johan Vos URL: https://git.openjdk.org/jfx17u/commit/eb9226375bdaa2edc0976d51f34337721ef0c9bb Stats:

[jfx17u] Integrated: 8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5

2024-03-08 Thread Johan Vos
8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5 - Commit messages: - Backport a7f6de8d13fd7a7e37ca80b28badcb24566153d8 Changes: https://git.openjdk.org/jfx17u/pull/186/files Webrev: https://webrevs.openjdk.org/?repo=jfx17u&pr=186&range=00 Issue: https://bugs.openjdk.

[jfx21u] Integrated: 8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 10:03:29 GMT, Johan Vos wrote: > 8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5 This pull request has now been integrated. Changeset: b0ed455c Author:Johan Vos URL: https://git.openjdk.org/jfx21u/commit/b0ed455c3bdb60b18427f568c609a89a8e88b4ed Stats:

[jfx21u] Integrated: 8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5

2024-03-08 Thread Johan Vos
8325258: Additional WebKit 617.1 fixes from WebKitGTK 2.42.5 - Commit messages: - Backport a7f6de8d13fd7a7e37ca80b28badcb24566153d8 Changes: https://git.openjdk.org/jfx21u/pull/52/files Webrev: https://webrevs.openjdk.org/?repo=jfx21u&pr=52&range=00 Issue: https://bugs.openjdk.or

[jfx17u] Integrated: 8324337: Cherry-pick WebKit 617.1 stabilization fixes

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 10:01:26 GMT, Johan Vos wrote: > 8324337: Cherry-pick WebKit 617.1 stabilization fixes This pull request has now been integrated. Changeset: 426f4eb0 Author:Johan Vos URL: https://git.openjdk.org/jfx17u/commit/426f4eb0001e9d1c747df0951446030c3d0adcec Stats: 12

[jfx17u] Integrated: 8324337: Cherry-pick WebKit 617.1 stabilization fixes

2024-03-08 Thread Johan Vos
8324337: Cherry-pick WebKit 617.1 stabilization fixes - Commit messages: - Backport 52840a17dee6a92fb9b1f3ff1b314b0ae228b95f Changes: https://git.openjdk.org/jfx17u/pull/185/files Webrev: https://webrevs.openjdk.org/?repo=jfx17u&pr=185&range=00 Issue: https://bugs.openjdk.org/bro

[jfx21u] Integrated: 8324337: Cherry-pick WebKit 617.1 stabilization fixes

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 09:29:55 GMT, Johan Vos wrote: > 8324337: Cherry-pick WebKit 617.1 stabilization fixes This pull request has now been integrated. Changeset: eec2c96b Author:Johan Vos URL: https://git.openjdk.org/jfx21u/commit/eec2c96b8d35b65d6090496451114b00c9ecd3ca Stats: 12

[jfx21u] RFR: 8324337: Cherry-pick WebKit 617.1 stabilization fixes

2024-03-08 Thread Johan Vos
8324337: Cherry-pick WebKit 617.1 stabilization fixes - Commit messages: - Backport 52840a17dee6a92fb9b1f3ff1b314b0ae228b95f Changes: https://git.openjdk.org/jfx21u/pull/51/files Webrev: https://webrevs.openjdk.org/?repo=jfx21u&pr=51&range=00 Issue: https://bugs.openjdk.org/brows

[jfx17u] Integrated: 8327177: macOS: wrong GlobalRef deleted in GlassMenu

2024-03-08 Thread Johan Vos
On Fri, 8 Mar 2024 09:09:37 GMT, Johan Vos wrote: > Hi all, > > This pull request contains a backport of commit 8114559e from the openjdk/jfx > repository. > > The commit being backported was authored by Johan Vos on 4 Mar 2024 and was > reviewed by Kevin Rushforth. > > Thanks! This pull re

[jfx17u] Integrated: 8327177: macOS: wrong GlobalRef deleted in GlassMenu

2024-03-08 Thread Johan Vos
Hi all, This pull request contains a backport of commit 8114559e from the openjdk/jfx repository. The commit being backported was authored by Johan Vos on 4 Mar 2024 and was reviewed by Kevin Rushforth. Thanks! - Commit messages: - Backport 8114559ed666fe272d238cdf1e531e5d8ec222

[jfx17u] Integrated: 8318387: Update GStreamer to 1.22.6

2024-03-08 Thread Johan Vos
On Thu, 7 Mar 2024 11:28:01 GMT, Johan Vos wrote: > Almost clean backport of JDK-8318386. > The original patch contains a fix to gstaacparse.c which is not in jfx17u > because it does not have JDK-8277309 (JDK-8277309) This pull request has now been integrated. Changeset: 34922457 Author:J

[jfx17u] Integrated: 8318614: Update WebKit to 617.1

2024-03-08 Thread Johan Vos
On Thu, 7 Mar 2024 11:32:29 GMT, Johan Vos wrote: > Hi all, > > This pull request contains a backport of commit > [ba79e081](https://github.com/openjdk/jfx/commit/ba79e081547b7f15697bfaaac42ec2de1971935a) > from the [openjdk/jfx](https://git.openjdk.org/jfx) repository. > > The commit being b

[jfx21u] Integrated: 8318614: Update WebKit to 617.1

2024-03-08 Thread Johan Vos
On Thu, 7 Mar 2024 11:03:42 GMT, Johan Vos wrote: > Hi all, > > This pull request contains a backport of commit > [ba79e081](https://github.com/openjdk/jfx/commit/ba79e081547b7f15697bfaaac42ec2de1971935a) > from the [openjdk/jfx](https://git.openjdk.org/jfx) repository. > > The commit being b

Re: [External] : Re: Q: Labeled.textTruncated Property (RFE)

2024-03-08 Thread Daniel Peintner
Andy, Marius, all, Thank you for your input and your insights. If I understand correctly we cannot purely do it with public methods for now (at least not for TableCells) This makes the argument to support https://github.com/openjdk/jfx/pull/1389 even stronger and maybe should be mentioned in the