@techee commented on this pull request.
> + if (sci_get_line_from_position(sci, pos_next) == line) + char_width = SSM(sci, SCI_POINTXFROMPOSITION, 0, pos_next) - x; > Another possible improvement would be doing something similar in the X > direction, although there's no actual issue with it. However, as is, if there > isn't enough room on the right, it would then popup on the left of the caret, > not under the character at the caret. It might be a nitpick, but it looks a > tiny bit off with caret in overwrite mode. Yet, I don't think the previous > code supported this. OK, found it. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3316#discussion_r1350743555 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/3316/review/1665179...@github.com>
