@techee commented on this pull request.


In src/symbols.c:

> +	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?

Yes it is (to be precise, with char_width=1, I wasn't sure if 0-width bounding box works alright in this case).

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.

I understand the current version problem but I still don't see the problem with char_width=0.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3316/review/1667483860@github.com>