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

2025-01-14 Thread Kevin Rushforth
On Fri, 22 Nov 2024 22:14:12 GMT, Martin Fox wrote: >> I'll take a look at it after Thanksgiving. >> >> Somewhat related questions (sorry if totally unrelated): >> >> Q1. shouldn't the input method be tied to the `Scene.focusOwner` property >> somehow? >> Q2. given the fact that there could be

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

2025-01-13 Thread Kevin Rushforth
On Fri, 13 Dec 2024 16:26:54 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

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

2024-12-17 Thread Andy Goryachev
On Fri, 13 Dec 2024 16:26:54 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

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

2024-12-13 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: RFR: 8288893: Popup and its subclasses cannot input text from InputMethod [v3]

2024-12-12 Thread Andy Goryachev
On Tue, 12 Nov 2024 15:08:11 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

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

2024-12-11 Thread Martin Fox
On Wed, 4 Dec 2024 14:39:55 GMT, Martin Fox wrote: >> modules/javafx.graphics/src/main/java/com/sun/javafx/scene/InputMethodStateManager.java >> line 48: >> >>> 46: * PopupWindow. >>> 47: */ >>> 48: public class InputMethodStateManager { >> >> All this feels overly complicated - >> >> Why d

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

2024-12-04 Thread Kevin Rushforth
On Wed, 4 Dec 2024 18:14:58 GMT, Kevin Rushforth wrote: >> One can extend `PopupWindow` and call the protected `setScene()`, so this is >> a valid scenario, I think. > > PopupWindow already specifies that subclasses should not do this: > > > Note to subclasses: the scene used by PopupWindow is

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

2024-12-04 Thread Kevin Rushforth
On Wed, 4 Dec 2024 15:36:00 GMT, Andy Goryachev wrote: >> In a PopupWindow the scene there is no API to set the scene. It's managed >> internally by the system and should not change over the lifetime of the >> popup window. I will take a second look at that code next week to verify. > > One can

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

2024-12-04 Thread Andy Goryachev
On Wed, 4 Dec 2024 14:39:52 GMT, Martin Fox wrote: >> modules/javafx.graphics/src/main/java/javafx/stage/PopupWindow.java line 578: >> >>> 576: // still monitoring owner events. >>> 577: Scene scene = getScene(); >>> 578: >>> SceneHelper.getInputMethodStateMa

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

2024-12-04 Thread Martin Fox
On Tue, 3 Dec 2024 22:06:48 GMT, Andy Goryachev wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Tweak to satisfy system test. > > modules/javafx.graphics/src/main/java/com/sun/javafx/scene/InputMethodStateManager.jav

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

2024-12-03 Thread Andy Goryachev
On Tue, 12 Nov 2024 15:08:11 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

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

2024-12-03 Thread Andy Goryachev
On Tue, 12 Nov 2024 15:08:11 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

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

2024-11-22 Thread Martin Fox
On Fri, 22 Nov 2024 19:51:18 GMT, Andy Goryachev wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Tweak to satisfy system test. > > I'll take a look at it after Thanksgiving. > > Somewhat related questions (sorry if

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

2024-11-22 Thread Andy Goryachev
On Tue, 12 Nov 2024 15:08:11 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

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: 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

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

2024-11-11 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

RFR: 8288893: Popup and its subclasses cannot input text from InputMethod

2024-11-11 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 contain a Comb