On Thu, 23 Feb 2023 09:20:26 GMT, John Hendrikx <jhendr...@openjdk.org> wrote:
> I think this class may benefit from a few tests that test with a very wide > caret, to see if positioning is what you'd expect in those cases as well. I > get the impression a lot of the code assumes a narrow caret (1 or 2 pixels) > and this is why we see constants like `0` and `1` in the code, and even > places where the caret width should be subtracted but isn't because it is > assumed to be `0`. Since caret width is hard coded and there is no direct way of changing the width, I believe we can proceed without adding tests with wide caret. ------------- PR: https://git.openjdk.org/jfx/pull/980