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

Ah… I feel it looks bad if cursor is in block mode (or overwrite) and the popup 
is placed on the left (when there isn't enough room on the right).  IMO, *one* 
of the solutions (preferably not the first) is better.  But I won't die on that 
hill :slightly_smiling_face: 

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

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

Reply via email to