Re: RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Andy Goryachev
On Mon, 24 Jun 2024 18:05:09 GMT, Jose Pereda wrote: > This PR considers the horizontal scroll events that can be generated on a > trackpad, on an horizontally sided `TabPane` control (top or bottom), adding > to the existing vertical scroll events from mouse wheel and trackpad, to > scroll it

Re: RFR: 8334657: Enable binary check

2024-06-24 Thread Phil Race
On Fri, 21 Jun 2024 12:50:45 GMT, Kevin Rushforth wrote: > This PR enables two warnings that are now (or soon will be) enabled in the > [jdk](https://github.com/openjdk/jdk) repo: > > 1. Binary file check -- this will alert the reviewers of a PR that the patch > being reviewed includes one or

Re: RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Andy Goryachev
On Mon, 24 Jun 2024 22:17:13 GMT, Kevin Rushforth wrote: > Do ScrollPane or ListView work this way already? Good point - both have the same issue (I am sure other controls as well, TableView, others did not check). Definitely deserves a separate ticket, possibly even an umbrella task (RTL nav

Re: RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Andy Goryachev
On Mon, 24 Jun 2024 22:17:13 GMT, Kevin Rushforth wrote: > I don't recommend doing that as part of this PR. That would be a noticeable > behavior change, and warrants more discussion we **are** changing the behavior already by changing the direction of the gesture, so I thought we should be ab

Re: RFR: 8311124: [Windows] User installed font 8281327 fix does not work for all cases

2024-06-24 Thread Kevin Rushforth
On Mon, 24 Jun 2024 22:01:09 GMT, Phil Race wrote: > The problem is due to per-user install fonts on windows having full paths so > they may be too long. > This is identical to the JDK fix that was done some time ago > https://github.com/openjdk/jdk/pull/13359 > I've tested that per-user fonts

Integrated: 8334731: GHA: build on macOS / aarch64

2024-06-24 Thread Kevin Rushforth
On Fri, 21 Jun 2024 15:22:57 GMT, Kevin Rushforth wrote: > GitHub Actions: add a macOS / aarch64 build using a macOS 14 test runner and > a macOS / aarch64 boot JDK. This is a brute force approach that duplicates > the job config (I don't want to spend the time to figure out how to > parameter

Re: RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Kevin Rushforth
On Mon, 24 Jun 2024 19:37:02 GMT, Andy Goryachev wrote: > 1. For Side.LEFT/RIGHT, reverse the direction. What we have now is > inconsistent with the movement of e.g. ListView (can be seen if using track > pad with the list view and TabPane in the Monkey Tester): I don't recommend doing that as

RFR: 8334900: IOOBE when adding data to a Series of a BarChart that already contains data

2024-06-24 Thread Markus Mack
This PR is a fix for another IOOBE that I discovered while working on #1476. The PR simplifies the code for adding a series that already contains data by adding the data points one-by-one. As far as I can see no attempt was previously made to optimize the bulk operation except for some trivial O

RFR: 8311124: [Windows] User installed font 8281327 fix does not work for all cases

2024-06-24 Thread Phil Race
The problem is due to per-user install fonts on windows having full paths so they may be too long. This is identical to the JDK fix that was done some time ago https://github.com/openjdk/jdk/pull/13359 I've tested that per-user fonts still work and so do the system-wide installed fonts. ---

Re: RFR: 8088923: IOOBE when adding duplicate categories to the BarChart [v5]

2024-06-24 Thread Markus Mack
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote: >> This PR provides the test case given in the JBS issue, and a simple fix for >> the index calculation when inserting data after previous data with duplicate >> categories. >> >> Also, I've added a comment to `BarChart`s javadoc, clarifying

Re: RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Kevin Rushforth
On Mon, 24 Jun 2024 19:37:02 GMT, Andy Goryachev wrote: > And, a question - does this change require a CSR, since we are changing the > behavior? Good question. Thinking out loud, if what this PR does is to change the existing behavior of top/bottom TabPane scrolling (e.g., horizontal scrollba

web rendering bug in JavaFX 22

2024-06-24 Thread Dan Howard
Hi all, I didn't try this with older JavaFX versions but I found a page that does not work if I render using the JavaFX browser. https://www.reonis.com/uatu/dungeoncraft/parts/iconmaker.html If you try this in firefox/chrome it works fine. Can anyone confirm? TIA

Re: RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Andy Goryachev
On Mon, 24 Jun 2024 18:05:09 GMT, Jose Pereda wrote: > This PR considers the horizontal scroll events that can be generated on a > trackpad, on an horizontally sided `TabPane` control (top or bottom), adding > to the existing vertical scroll events from mouse wheel and trackpad, to > scroll it

Re: RFR: 8088923: IOOBE when adding duplicate categories to the BarChart [v5]

2024-06-24 Thread Andy Goryachev
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote: >> This PR provides the test case given in the JBS issue, and a simple fix for >> the index calculation when inserting data after previous data with duplicate >> categories. >> >> Also, I've added a comment to `BarChart`s javadoc, clarifying

Integrated: 8334739: XYChart and (Stacked)AreaChart properties return incorrect beans

2024-06-24 Thread Markus Mack
On Fri, 21 Jun 2024 16:11:52 GMT, Markus Mack wrote: > This PR makes the `XYChart` axes properties return the correct parent XYChart > object. This pull request has now been integrated. Changeset: 17c2dba0 Author:Markus Mack Committer: Andy Goryachev URL: https://git.openjdk.org/j

Integrated: 8088923: IOOBE when adding duplicate categories to the BarChart

2024-06-24 Thread Markus Mack
On Wed, 19 Jun 2024 10:21:52 GMT, Markus Mack wrote: > This PR provides the test case given in the JBS issue, and a simple fix for > the index calculation when inserting data after previous data with duplicate > categories. > > Also, I've added a comment to `BarChart`s javadoc, clarifying the

RFR: 8334874: Horizontal scroll events from touch pads should scroll the TabPane tabs

2024-06-24 Thread Jose Pereda
This PR considers the horizontal scroll events that can be generated on a trackpad, on an horizontally sided `TabPane` control (top or bottom), adding to the existing vertical scroll events from mouse wheel and trackpad, to scroll its tabs. Therefore, scrolling a `TabPane` will behave in the sa

[jfx21u] RFR: 8329705: Add missing Application thread checks to platform specific a11y methods

2024-06-24 Thread Johan Vos
8329705: Add missing Application thread checks to platform specific a11y methods - Commit messages: - Backport 0eb4d7196099d817cc6467985b882242845bdd2e Changes: https://git.openjdk.org/jfx21u/pull/61/files Webrev: https://webrevs.openjdk.org/?repo=jfx21u&pr=61&range=00 Issue: ht

[jfx17u] RFR: 8329705: Add missing Application thread checks to platform specific a11y methods

2024-06-24 Thread Johan Vos
8329705: Add missing Application thread checks to platform specific a11y methods - Commit messages: - Backport 0eb4d7196099d817cc6467985b882242845bdd2e Changes: https://git.openjdk.org/jfx17u/pull/194/files Webrev: https://webrevs.openjdk.org/?repo=jfx17u&pr=194&range=00 Issue:

[jfx17u] Integrated: 8301312: Create implementation of NSAccessibilityButton protocol

2024-06-24 Thread Johan Vos
On Mon, 24 Jun 2024 13:47:54 GMT, Johan Vos wrote: > 8301312: Create implementation of NSAccessibilityButton protocol This pull request has now been integrated. Changeset: cf66048e Author:Johan Vos URL: https://git.openjdk.org/jfx17u/commit/cf66048eb7519a2c652ccf388218c385e8820151 S

Re: RFR: 8327255: javac lint warnings: removal, missing-explicit-ctor [v2]

2024-06-24 Thread Ambarish Rapte
On Sat, 22 Jun 2024 13:37:30 GMT, Kevin Rushforth wrote: >> This PR updates `build.gradle` to define javac lint options for three >> different types of java compilation tasks: sdk classes, test classes >> (including shims), and tool classes (including JSLC). The defaults for these >> three gro

Re: RFR: 8334657: Enable binary check

2024-06-24 Thread Ambarish Rapte
On Fri, 21 Jun 2024 12:50:45 GMT, Kevin Rushforth wrote: > This PR enables two warnings that are now (or soon will be) enabled in the > [jdk](https://github.com/openjdk/jdk) repo: > > 1. Binary file check -- this will alert the reviewers of a PR that the patch > being reviewed includes one or

Re: RFR: 8088923: IOOBE when adding duplicate categories to the BarChart [v5]

2024-06-24 Thread Andy Goryachev
On Sat, 22 Jun 2024 10:58:53 GMT, Markus Mack wrote: > I can reproduce this, looks unrelated to the issue this PR is trying to fix. Created https://bugs.openjdk.org/browse/JDK-8334873 - PR Comment: https://git.openjdk.org/jfx/pull/1476#issuecomment-2186856495

Re: RFR: 8088923: IOOBE when adding duplicate categories to the BarChart [v5]

2024-06-24 Thread Andy Goryachev
On Sat, 22 Jun 2024 10:55:28 GMT, Markus Mack wrote: >> This PR provides the test case given in the JBS issue, and a simple fix for >> the index calculation when inserting data after previous data with duplicate >> categories. >> >> Also, I've added a comment to `BarChart`s javadoc, clarifying

Re: RFR: 8088923: IOOBE when adding duplicate categories to the BarChart [v4]

2024-06-24 Thread Andy Goryachev
On Sat, 22 Jun 2024 10:52:42 GMT, Markus Mack wrote: >> modules/javafx.controls/src/main/java/javafx/scene/chart/BarChart.java line >> 227: >> >>> 225: var uniqueCategories = new HashSet(); >>> 226: for (var entry : seriesCategoryMap.entrySet()) { >>> 227:

[jfx17u] RFR: 8301312: Create implementation of NSAccessibilityButton protocol

2024-06-24 Thread Johan Vos
8301312: Create implementation of NSAccessibilityButton protocol - Commit messages: - Backport 05548ace27f099a5ce152e3fb9efb4d22da683c8 Changes: https://git.openjdk.org/jfx17u/pull/193/files Webrev: https://webrevs.openjdk.org/?repo=jfx17u&pr=193&range=00 Issue: https://bugs.ope