On Thu, 26 Mar 2026 21:00:27 GMT, Andy Goryachev <[email protected]> wrote:
>> +1 for a follow up. I think not all final values that are later set in the >> translate etc. are snapped as of now. So we probably need to catch all >> locations at once. > > Do I understand you correctly - the issue is still seen with the fix at > fractional scales? If so, then it would make sense to fix it in this PR. Yes, the issue of the missing caret still exists at some fractional scales. However, I would say that this is a separate issue, more general to the whole skin implementation. So there are two issues here, which cause the same problem in some scenarios, but the causes and potential fixes don't really have a lot to do with each other. This one just repairs a small part that was broken during and earlier PR. To fix the caret issue on fractional scales, that would require a more investigation. The JBS issue focuses on the broken caret width calculation, not the missing caret behavior so I think splitting this up into a separate issue makes more sense ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2128#discussion_r2997866559
