@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;
Maybe it was mentioned somewhere in the previous conversation but what's the point of calculating `char_width`? Couldn't the rect width just be always `1`? Clearly the height is important so the menu is below the line but with width `1` the menu pops up from the gap between characters which looks fine and not unexpected IMO.