Re: RFR: 8306707: Support pluggable image loading via javax.imageio [v11]

2024-11-12 Thread Jayathirth D V
On Fri, 1 Nov 2024 16:59:50 GMT, Kevin Rushforth wrote: > 2. This patch causes a failure in one of our closed white-box tests, > which depends on internal API that has changed. It will be easy for us to fix > it, but it will mean that down the road, when this is ready to go in, we will > n

Re: Unconsumed event handlers

2024-11-12 Thread John Hendrikx
On 12/11/2024 18:31, Andy Goryachev wrote: I am not sure this is the best solution, since it does not solve the problem of multiple actors adding their event handlers. I do like the idea of prioritized event handlers, because it solves the problem **reliably**. I think there is no way arou

Re: Unconsumed event handlers

2024-11-12 Thread Martin Fox
John, The issue occurs inside the event dispatchers. An event is passed into dispatchEvent which may return a separate event. What state is carried over from the incoming event to the outgoing event? If there are unconsumed event handlers attached to the incoming event are they also attached to

Re: RFR: 8301121: RichTextArea Control (Incubator) [v41]

2024-11-12 Thread Andy Goryachev
> Incubating a new feature - rich text control, **RichTextArea**, intended to > bridge the functional gap with Swing and its StyledEditorKit/JEditorPane. The > main design goal is to provide a control that is complete enough to be useful > out-of-the box, as well as open to extension by the appl

Re: RFR: 8301121: RichTextArea Control (Incubator) [v40]

2024-11-12 Thread Andy Goryachev
> Incubating a new feature - rich text control, **RichTextArea**, intended to > bridge the functional gap with Swing and its StyledEditorKit/JEditorPane. The > main design goal is to provide a control that is complete enough to be useful > out-of-the box, as well as open to extension by the appl

Integrated: 8091673: Public focus traversal API for use in custom controls

2024-11-12 Thread Andy Goryachev
On Wed, 6 Nov 2024 17:49:59 GMT, Andy Goryachev wrote: > Public focus traversal API for use in custom controls. > > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/FocusTraversal/FocusTraversal-v3.md > > This is a lightweight change that only adds the public API for focus > travers

Re: RFR: 8344067: TableCell indices may not match the TableRow index

2024-11-12 Thread Andy Goryachev
On Tue, 12 Nov 2024 21:11:56 GMT, Marius Hanl wrote: > This PR fixes a bug where the `TableCell` indices can be outdated (not > synchronized) with the `TableRow` index. > > What normally happens is: > - Index is changed: Cell update is requested when the row is empty (otherwise > noop) > - Ite

RFR: 8344067: TableCell indices can be outdated with the TableRow index

2024-11-12 Thread Marius Hanl
This PR fixes a bug where the `TableCell` indices can be outdated (not synchronized) with the `TableRow` index. What normally happens is: - Index is changed: Cell update is requested when the row is empty (otherwise noop) - Item is changed: Cell update is requested, which will now update the ind

Re: CFV: New OpenJFX Reviewer: Lukasz Kostyra

2024-11-12 Thread Philip Race
Vote: yes -phil.

Re: Prioritized event handlers

2024-11-12 Thread Andy Goryachev
I'd like to understand that too, though the question here might be a slightly different topic (maybe for a separate mail thread). The way I look at the problem of using controls as part of other controls (e.g. a TextField inside a Spinner) is that the inner is no longer a fully functional contr

Re: Focus delegation API

2024-11-12 Thread Michael Strauß
Responding to Nir's questions in this thread. > I'd like to understand the focus and event handling problem better. If I have > a focused TextField, all key events go to it. If I have a Spinner, which is a > TextField with 2 Buttons, it is focused as a single unit and consumes key > events whet

Re: Prioritized event handlers

2024-11-12 Thread Michael Strauß
I would encourage to continue this discussion in the focus delegation thread, not in the prioritized event handlers thread. On Tue, Nov 12, 2024 at 8:13 PM Nir Lisker wrote: > > I'd like to understand the focus and event handling problem better. If I have > a focused TextField, all key events

Re: Prioritized event handlers

2024-11-12 Thread John Hendrikx
On 12/11/2024 19:46, Nir Lisker wrote: I'd like to understand the focus and event handling problem better. If I have a focused TextField, all key events go to it. If I have a Spinner, which is a TextField with 2 Buttons, it is focused as a single unit and consumes key events whether they are ai

Re: Unconsumed event handlers

2024-11-12 Thread John Hendrikx
Hi Martin, I'm still unsure how this exactly is supposed to go wrong. Whenever you dispatch a new event, you should always follow-up and check if it was consumed after the dispatch finishes.  If it is, then you mark the triggering event also as consumed.  If this is done consistently everywhe

Re: CFV: New OpenJFX Reviewer: Lukasz Kostyra

2024-11-12 Thread John Hendrikx
Vote: YES On 12/11/2024 17:22, Kevin Rushforth wrote: I hereby nominate Lukasz Kostyra [1] to OpenJFX Reviewer. Lukasz is a member of JavaFX team at Oracle, who has contributed 35 commits [2] to OpenJFX. Lukasz consistently participates in code reviews of various pull requests, especially in

Re: Prioritized event handlers

2024-11-12 Thread Nir Lisker
I'd like to understand the focus and event handling problem better. If I have a focused TextField, all key events go to it. If I have a Spinner, which is a TextField with 2 Buttons, it is focused as a single unit and consumes key events whether they are aimed at the text field or the buttons (I ass

Re: Unconsumed event handlers

2024-11-12 Thread Martin Fox
(Here’s hoping this e-mail makes it out to the list. If this is an ongoing problem I might have to switch to using my gmail address.) I’m writing up some notes to improve the documentation on event dispatch. There seems to be only one rule governing dispatchers and that is “return null from dis

Re: Prioritized event handlers

2024-11-12 Thread Kevin Rushforth
I think this is a good discussion to continue. I have a a couple quick comments: The first question I would like to resolve is to determine whether the problem exists globally, or only at the controls level.  If even once scenario exists that does not involve controls, we must find a solution

Re: RFR: 8313424: JavaFX controls in the title bar [v27]

2024-11-12 Thread Andy Goryachev
On Sat, 9 Nov 2024 07:45:19 GMT, Michael Strauß wrote: >> Implementation of [`EXTENDED` stage >> style](https://gist.github.com/mstr2/0befc541ee7297b6db2865cc5e4dbd09). > > Michael Strauß has updated the pull request with a new target base due to a > merge or a rebase. The pull request now cont

Re: Unconsumed event handlers

2024-11-12 Thread Andy Goryachev
I am not sure this is the best solution, since it does not solve the problem of multiple actors adding their event handlers. I do like the idea of prioritized event handlers, because it solves the problem *reliably*. I think there is no way around it - we need different priorities. It may be a

Aw: CFV: New OpenJFX Committer: Jayathirth D V

2024-11-12 Thread Marius Hanl
Vote: Yes.-- MariusAm 08.11.24, 14:50 schrieb Kevin Rushforth : I hereby nominate Jayathirth D V [1] to OpenJFX Committer. Jay is a member of the JavaFX team at Oracle who has contributed 12 commits [2] to OpenJFX. He has contributed an additional 5 commits to the OpenJFX jfx-tests repo [3]. Vo

Re: CFV: New OpenJFX Reviewer: Lukasz Kostyra

2024-11-12 Thread Nir Lisker
Vote: YES On Tue, Nov 12, 2024 at 6:24 PM Andy Goryachev wrote: > Vote: YES > > > > -andy > > > > > > > > *From: *openjfx-dev on behalf of Kevin > Rushforth > *Date: *Tuesday, November 12, 2024 at 08:23 > *To: *openjfx-dev , Lukasz Kostyra < > lukasz.kost...@oracle.com> > *Subject: *CFV: New O

Re: CFV: New OpenJFX Reviewer: Lukasz Kostyra

2024-11-12 Thread Andy Goryachev
Vote: YES -andy From: openjfx-dev on behalf of Kevin Rushforth Date: Tuesday, November 12, 2024 at 08:23 To: openjfx-dev , Lukasz Kostyra Subject: CFV: New OpenJFX Reviewer: Lukasz Kostyra I hereby nominate Lukasz Kostyra [1] to OpenJFX Reviewer. Lukasz is a member of JavaFX team at Oracl

Re: Prioritized event handlers

2024-11-12 Thread Andy Goryachev
Thank you Michael for answering my questions! I get from your answers that: 1. the priorities are still needed, in one form or the other. Adding a different type of the EH (ifUnconsumed) seems to me like a different priority. 2. the problem seems to exist only at the controls level - nothing wa

Re: CFV: New OpenJFX Reviewer: Lukasz Kostyra

2024-11-12 Thread Kevin Rushforth
Vote: YES -- Kevin On 11/12/2024 8:22 AM, Kevin Rushforth wrote: I hereby nominate Lukasz Kostyra [1] to OpenJFX Reviewer.

CFV: New OpenJFX Reviewer: Lukasz Kostyra

2024-11-12 Thread Kevin Rushforth
I hereby nominate Lukasz Kostyra [1] to OpenJFX Reviewer. Lukasz is a member of JavaFX team at Oracle, who has contributed 35 commits [2] to OpenJFX. Lukasz consistently participates in code reviews of various pull requests, especially in the graphics area. Votes are due by Nov 26, 2024 at 17

Re: RFR: 8288893: Popup and its subclasses cannot input text from InputMethod [v3]

2024-11-12 Thread Martin Fox
> Input methods don’t work for text controls inside Popups. This PR fixes that. > > Some background: > > A PopupWindow always has an owner. The owner of the first Popup is a standard > Window; I’ll refer to that as the root window. But a popup can show another > popup (for example, a popup may

Re: My bug report seems to have been lost

2024-11-12 Thread Kevin Rushforth
This is now visible here: https://bugs.openjdk.org/browse/JDK-8344037 -- Kevin On 11/12/2024 7:07 AM, PavelTurk wrote: Hello everyone, I submitted a bug report on November 6, but I still haven’t received any notification about it. Review ID - 9077800. The bug was about Font.loadFont() B

Re: RFR: 8288893: Popup and its subclasses cannot input text from InputMethod [v2]

2024-11-12 Thread Martin Fox
On Mon, 11 Nov 2024 19:34:04 GMT, Martin Fox wrote: >> Input methods don’t work for text controls inside Popups. This PR fixes that. >> >> Some background: >> >> A PopupWindow always has an owner. The owner of the first Popup is a >> standard Window; I’ll refer to that as the root window. But

My bug report seems to have been lost

2024-11-12 Thread PavelTurk
Hello everyone, I submitted a bug report on November 6, but I still haven’t received any  notification about it. Review ID - 9077800. The bug was about Font.loadFont() Best regards, Pavel

Re: Unconsumed event handlers

2024-11-12 Thread John Hendrikx
I think `discardUnconsumedEventHandlers` is still a bit unpredictable, as it would depend on handler order. The user handler would need to be called *after* Skin/Behavior handlers for it to be able to always discard any ifUnconsumed callbacks.  If the Skin is replaced, then it will re-register