Re: RFR: 8347305: RichTextArea Follow-up [v2]

2025-01-10 Thread Kevin Rushforth
On Fri, 10 Jan 2025 16:24:05 GMT, Andy Goryachev wrote: >> The changes: >> >> - minor fixes in the behavior document (DELETE_WORD_PREVIOUS) >> - Default width of RTA is less than half the size of TextArea (176 vs 478 on >> my Windows 11 system). Earlier the height was also too small, but you >

Re: RFR: 8347305: RichTextArea Follow-up [v2]

2025-01-10 Thread Andy Goryachev
> The changes: > > - minor fixes in the behavior document (DELETE_WORD_PREVIOUS) > - Default width of RTA is less than half the size of TextArea (176 vs 478 on > my Windows 11 system). Earlier the height was also too small, but you already > fixed that. > > The following is not an issue (works

Re: RFR: 8347305: RichTextArea Follow-up

2025-01-10 Thread Kevin Rushforth
On Fri, 10 Jan 2025 15:48:43 GMT, Andy Goryachev wrote: > The changes: > > - minor fixes in the behavior document (DELETE_WORD_PREVIOUS) > - Default width of RTA is less than half the size of TextArea (176 vs 478 on > my Windows 11 system). Earlier the height was also too small, but you already

Re: RFR: 8347305: RichTextArea Follow-up

2025-01-10 Thread Andy Goryachev
On Fri, 10 Jan 2025 16:07:38 GMT, Kevin Rushforth wrote: > It might just be an issue with the demo, I see "use content width" option enabled, this is not the right use case. When the RTA is constrained, the useContentWidth should be off. - PR Comment: https://git.openjdk.org/jfx/

Re: RFR: 8347305: RichTextArea Follow-up

2025-01-10 Thread Kevin Rushforth
On Fri, 10 Jan 2025 15:48:43 GMT, Andy Goryachev wrote: > The following is not an issue (works as designed): > > * When running the demos, the current line highlight overlaps and obscures > focus outline on right side (verified on Windows and Linux). It might just be an issue with the demo, be

RFR: 8347305: RichTextArea Follow-up

2025-01-10 Thread Andy Goryachev
The changes: - minor fixes in the behavior document (DELETE_WORD_PREVIOUS) - Default width of RTA is less than half the size of TextArea (176 vs 478 on my Windows 11 system). Earlier the height was also too small, but you already fixed that. The following is not an issue (works as designed): -