search

rgieseke / textredux

Text-based interfaces for Textadept
http://rgieseke.github.io/textredux/
Other
59 stars 11 forks source link

Marker margin width based on line number margin width #85

Open jxzwp opened 1 year ago

jxzwp commented 1 year ago

The width of margin 2, the marker margin is based upon the width of margin 1, the line number margin. Which doesn't make much sense and results in it being wider than needed.

https://github.com/rgieseke/textredux/blob/b7900c77e1604ed0f0c3a4b9c0b0776e89bb7b2a/core/buffer.lua#L491

I think the line above should be 

target.margin_width_n[2] = not CURSES and 4 or 1

That's the default in TA 11 and TA 12 see

https://github.com/orbitalquark/textadept/blob/2dc0609b81018fa52f75084ec6c22b59c5771e6b/init.lua#L171

Thanks.

rgieseke commented 1 year ago

Good question. I think the original intention was to somehow hide the line number (but I might mis-remember). This looks indeed odd though.