@b4n 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;

This is with the current state of this PR right, not with just `char_width=0`, 
is it?

The problem with the current version is only if you have wrapping enabled and 
it wraps after the next character (e.g. if wrapping wrapped right after the `i` 
(not even having the space after). What you'd see is the popup on the far left 
of the editor.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3316#discussion_r1352334264
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3316/review/1667449...@github.com>

Reply via email to