Re: RFR: JDK-8186188: TableColumHeader: initial auto-size broken if has graphic [v4]

2024-03-22 Thread Andy Goryachev
On Wed, 20 Mar 2024 19:53:40 GMT, Marius Hanl wrote: >> This PR fixes the issue that the initial column autosizing is wrong under >> some circumstances. >> The following things will break the initial autosizing: >> - Bold Column text (that is where I initially found this problem) >> - Another

Re: RFR: JDK-8328750: [TestBug] Improve Stub Font Support

2024-03-22 Thread Andy Goryachev
On Thu, 21 Mar 2024 22:06:42 GMT, Marius Hanl wrote: > In https://github.com/openjdk/jfx/pull/1405, I identified some shortcomings > of the stub font implementation. As I don't want to clutter the PR with that, > I decided to cherrypick the improvements I did to a new ticket and PR. > > The

Re: RFC: Screen capturing utility (for failed headful tests)

2024-03-22 Thread Kevin Rushforth
I like this idea in general. It would be a good thing to consider for a future test sprint. I have a few high-level questions on the specifics: 1. I've not used JUnit5 TestWatcher at all, but it seems like it might do what we want. Does anyone else have practical experience with it? For cases

RFC: Screen capturing utility (for failed headful tests)

2024-03-22 Thread Andy Goryachev
Dear colleagues: What do you think of the following idea: The Problem === Automated headful tests might fail intermittently or on specific platform(s). After the fact, it's nearly impossible to understand why this happened. Solution Implement a JUnit5 TestWatcher which

Re: RFR: 8328718: Remove unused imports in javafx.controls

2024-03-22 Thread Nir Lisker
On Thu, 21 Mar 2024 18:47:24 GMT, Andy Goryachev wrote: > Using Eclipse IDE to remove unused imports in javafx.controls and update the > copyright year to 2024. Using wildcard for more than 10 static imports. > > This is a trivial change, 1 reviewer is probably enough. I can, but it will

Re: RFR: 8328718: Remove unused imports in javafx.controls

2024-03-22 Thread Andy Goryachev
On Thu, 21 Mar 2024 18:47:24 GMT, Andy Goryachev wrote: > Using Eclipse IDE to remove unused imports in javafx.controls and update the > copyright year to 2024. Using wildcard for more than 10 static imports. > > This is a trivial change, 1 reviewer is probably enough. @Maran23 @nlisker

Re: RFR: 8316372: Monkey Tester Application Part 3 [v3]

2024-03-22 Thread Andy Goryachev
> Further changes to the MonkeyTester application: > > - remember split pane divider ✔ > - use 'private' instead of 'protected' in many cases ✔ > - added more scripts to the 'writing systems' text sample ✔ > - added RTL window control menu ✔ > - added embedded swing/fx in tools ✔ > - added copy

Re: RFR: 8316372: Monkey Tester Application Part 3 [v2]

2024-03-22 Thread Andy Goryachev
On Fri, 22 Mar 2024 13:00:02 GMT, Karthik P K wrote: > * In all the pages, under Region option, if we select MAX_VALUE for Min > Height or Min Width, the application hangs or whole window becomes white. I > observed this issue if we select MIN_VALUE or POSITIVE_INFINITY as well. works as

RFR: 8328820: Remove unused imports in javafx.swing

2024-03-22 Thread Andy Goryachev
Removed unused imports in javafx.swing (1 change) module. See the discussion in [JDK-8289379](https://bugs.openjdk.org/browse/JDK-8289379) - Commit messages: - 8328820: Remove unused imports in javafx.swing Changes: https://git.openjdk.org/jfx/pull/1428/files Webrev:

RFR: 8267565: Support "@1x" and "@3x" high-density image naming convention

2024-03-22 Thread drmarmac
This PR extends the range of hi-res images that are loaded via naming convention, now including scale factors higher than "@2x". Supporting these is already being [recommended](https://developer.apple.com/design/human-interface-guidelines/images#Best-practices) for some platforms. I tested

RFR: 8328811: Fix missing @Overrides in demos

2024-03-22 Thread Andy Goryachev
Fixing missing @Overrides in demo apps. This is still a trivial change since all the spots are identified by the IDE. - Commit messages: - part 3 - part 2 - part 1 Changes: https://git.openjdk.org/jfx/pull/1426/files Webrev: https://webrevs.openjdk.org/?repo=jfx=1426=00

RFR: 8328754: Fix missing @Overrides in test

2024-03-22 Thread Andy Goryachev
Fixing missing @ OVERRIDES in tests. This is still a trivial change since all the spots are identified by the IDE. - Commit messages: - 8328754: Fix missing @Overrides in test Changes: https://git.openjdk.org/jfx/pull/1427/files Webrev:

RFR: 8328818: Fix Decora JSL to remove missing @Override warnings

2024-03-22 Thread Andy Goryachev
Updating the JSL files to remove the "missing @ override" warning. - Commit messages: - 8328818: Fix Decora JSL to remove missing @Override warnings Changes: https://git.openjdk.org/jfx/pull/1425/files Webrev: https://webrevs.openjdk.org/?repo=jfx=1425=00 Issue:

Re: RFR: 8328749: Remove unused imports in javafx.web

2024-03-22 Thread Andy Goryachev
On Thu, 21 Mar 2024 21:57:46 GMT, Andy Goryachev wrote: > Using Eclipse IDE to remove unused imports **javafx.web** module, and update > the copyright year to 2024. Using wildcard for more than 10 static imports. > > > -- > > This is a trivial change, 1 reviewer is probably enough. removed

Re: RFR: 8328749: Remove unused imports in javafx.web [v2]

2024-03-22 Thread Andy Goryachev
> Using Eclipse IDE to remove unused imports **javafx.web** module, and update > the copyright year to 2024. Using wildcard for more than 10 static imports. > > > -- > > This is a trivial change, 1 reviewer is probably enough. Andy Goryachev has updated the pull request incrementally with one

RFR: 8328749: Remove unused imports in javafx.swing, javafx.web

2024-03-22 Thread Andy Goryachev
Using Eclipse IDE to remove unused imports in **javafx.swing** and **javafx.web** modules, and update the copyright year to 2024. Using wildcard for more than 10 static imports. -- This is a trivial change, 1 reviewer is probably enough. - Commit messages: - 8328749: Remove

Re: RFR: 8328749: Remove unused imports in javafx.swing, javafx.web

2024-03-22 Thread Kevin Rushforth
On Fri, 22 Mar 2024 15:52:02 GMT, Andy Goryachev wrote: > I suppose I'd need to extract swing into a separate PR, right @kevinrushforth > ? Yes. I was just getting ready to write this: It looks like I spoke too soon when I said it was OK to combine the change to `javafx.swing` into this PR.

Re: RFR: 8328749: Remove unused imports in javafx.swing, javafx.web

2024-03-22 Thread Andy Goryachev
On Thu, 21 Mar 2024 21:57:46 GMT, Andy Goryachev wrote: > Using Eclipse IDE to remove unused imports in **javafx.swing** and > **javafx.web** modules, and update the copyright year to 2024. Using wildcard > for more than 10 static imports. > > > -- > > This is a trivial change, 1 reviewer

Re: RFR: 8328749: Remove unused imports in javafx.swing, javafx.web

2024-03-22 Thread Kevin Rushforth
On Thu, 21 Mar 2024 21:57:46 GMT, Andy Goryachev wrote: > Using Eclipse IDE to remove unused imports in **javafx.swing** and > **javafx.web** modules, and update the copyright year to 2024. Using wildcard > for more than 10 static imports. > > > -- > > This is a trivial change, 1 reviewer

Re: RFR: 8328752: Fix missing @Overrides in javafx.web

2024-03-22 Thread Kevin Rushforth
On Thu, 21 Mar 2024 23:48:40 GMT, Andy Goryachev wrote: > Fix missing @Overrides in **javafx.web**. > > This is still a trivial change since all the spots are identified by the IDE. There are special considerations when touching files under

RFR: 8328752: Fix missing @Overrides in javafx.web

2024-03-22 Thread Andy Goryachev
Fix missing @Overrides in **javafx.web**. This is still a trivial change since all the spots are identified by the IDE. - Commit messages: - part 2 - 8328752: Fix missing @Overrides in javafx.web Changes: https://git.openjdk.org/jfx/pull/1424/files Webrev:

Re: RFR: 8316372: Monkey Tester Application Part 3 [v2]

2024-03-22 Thread Karthik P K
On Wed, 20 Mar 2024 23:56:50 GMT, Andy Goryachev wrote: >> Further changes to the MonkeyTester application: >> >> - remember split pane divider ✔ >> - use 'private' instead of 'protected' in many cases ✔ >> - added more scripts to the 'writing systems' text sample ✔ >> - added RTL window