Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v3]

2024-02-23 Thread Kevin Rushforth
On Fri, 2 Feb 2024 18:43:19 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v3]

2024-02-05 Thread Andy Goryachev
On Fri, 2 Feb 2024 18:43:19 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v3]

2024-02-02 Thread Martin Fox
On Fri, 2 Feb 2024 18:43:19 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v3]

2024-02-02 Thread Martin Fox
> In the Mac glass code the presence of "marked" text (which is tracked in the > nsAttrBuffer) signals that an IME is active. In this state the current code > assumes that when NSTextInputContext handles a `keyDown:` it will either > generate a call to `insertText:replacementRange:` or one of

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-02-02 Thread Kevin Rushforth
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-02-02 Thread Martin Fox
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-02-02 Thread Kevin Rushforth
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-02-02 Thread Andy Goryachev
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-02-02 Thread Andy Goryachev
On Tue, 30 Jan 2024 20:43:33 GMT, Martin Fox wrote: >>> The order in which they get resolved doesn't matter. >> >> would it be possible to get >> [JDK-8320912](https://bugs.openjdk.org/browse/JDK-8320912) integrated first >> then? It seems the scenario it addresses is narrower, and it will

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-02-02 Thread Kevin Rushforth
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-30 Thread Andy Goryachev
On Tue, 30 Jan 2024 20:43:33 GMT, Martin Fox wrote: > I submitted PR thank you! - PR Comment: https://git.openjdk.org/jfx/pull/1351#issuecomment-1917861792

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-30 Thread Martin Fox
On Tue, 30 Jan 2024 17:33:06 GMT, Andy Goryachev wrote: >>> > [JDK-8320912](https://bugs.openjdk.org/browse/JDK-8320912). I put >>> > together a fix for that bug but decided not to roll it into this PR. >>> >>> Are these two bugs independent? Should JDK-8320912 be resolved first? >> >> They

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-30 Thread Andy Goryachev
On Tue, 30 Jan 2024 17:21:37 GMT, Martin Fox wrote: > The order in which they get resolved doesn't matter. would it be possible to get [JDK-8320912](https://bugs.openjdk.org/browse/JDK-8320912) integrated first then? It seems the scenario it addresses is narrower, and it will be easier to

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-30 Thread Martin Fox
On Tue, 30 Jan 2024 15:56:04 GMT, Andy Goryachev wrote: > > [JDK-8320912](https://bugs.openjdk.org/browse/JDK-8320912). I put together > > a fix for that bug but decided not to roll it into this PR. > > Are these two bugs independent? Should JDK-8320912 be resolved first? They are

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-30 Thread Andy Goryachev
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-30 Thread Martin Fox
On Mon, 29 Jan 2024 19:04:10 GMT, Andy Goryachev wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> When IM enabled state changes we dismiss the IM window. > > ... and scenarios described in JDK-8088172 and JDK-8089803

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-29 Thread Martin Fox
On Tue, 30 Jan 2024 00:57:53 GMT, Martin Fox wrote: > Currently it commits a zero-length string (which appears to be a no-op) and > leaves the composition buffer untouched. Correcting myself: it does clear the composition buffer. But the zero-length commit doesn't seem to be doing what it

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-29 Thread Martin Fox
On Mon, 29 Jan 2024 19:04:10 GMT, Andy Goryachev wrote: >> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> When IM enabled state changes we dismiss the IM window. > > ... and scenarios described in JDK-8088172 and JDK-8089803

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-29 Thread Andy Goryachev
On Sat, 27 Jan 2024 19:19:48 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline

2024-01-29 Thread Andy Goryachev
On Sat, 27 Jan 2024 19:15:01 GMT, Martin Fox wrote: >> In the Mac glass code the presence of "marked" text (which is tracked in the >> nsAttrBuffer) signals that an IME is active. In this state the current code >> assumes that when NSTextInputContext handles a `keyDown:` it will either >>

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline

2024-01-27 Thread Martin Fox
On Fri, 26 Jan 2024 17:32:02 GMT, Martin Fox wrote: > In the Mac glass code the presence of "marked" text (which is tracked in the > nsAttrBuffer) signals that an IME is active. In this state the current code > assumes that when NSTextInputContext handles a `keyDown:` it will either >

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline [v2]

2024-01-27 Thread Martin Fox
> In the Mac glass code the presence of "marked" text (which is tracked in the > nsAttrBuffer) signals that an IME is active. In this state the current code > assumes that when NSTextInputContext handles a `keyDown:` it will either > generate a call to `insertText:replacementRange:` or one of

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline

2024-01-27 Thread Kevin Rushforth
On Fri, 26 Jan 2024 17:32:02 GMT, Martin Fox wrote: > In the Mac glass code the presence of "marked" text (which is tracked in the > nsAttrBuffer) signals that an IME is active. In this state the current code > assumes that when NSTextInputContext handles a `keyDown:` it will either >

Re: RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline

2024-01-26 Thread Kevin Rushforth
On Fri, 26 Jan 2024 17:32:02 GMT, Martin Fox wrote: > In the Mac glass code the presence of "marked" text (which is tracked in the > nsAttrBuffer) signals that an IME is active. In this state the current code > assumes that when NSTextInputContext handles a `keyDown:` it will either >

RFR: 8301900: TextArea: Committing text with ENTER in an IME window inserts newline

2024-01-26 Thread Martin Fox
In the Mac glass code the presence of "marked" text (which is tracked in the nsAttrBuffer) signals that an IME is active. In this state the current code assumes that when NSTextInputContext handles a `keyDown:` it will either generate a call to `insertText:replacementRange:` or one of the