Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The blank space to the left of the line numbers #1247

Closed
philiprbrenan opened this issue Sep 28, 2016 · 11 comments
Closed

The blank space to the left of the line numbers #1247

philiprbrenan opened this issue Sep 28, 2016 · 11 comments

Comments

@philiprbrenan
Copy link

numberspacing

Please consider removing the blank space to the left of the line numbers. Thanks!

@b4n
Copy link
Member

b4n commented Sep 28, 2016

I don't see that here. But how many lines does your file have? The line number margin is set as wide as necessary to display any of the line numbers in the file. Your screenshot looks just like would a file with at least 1000 lines, perhaps 10000.

Otherwise, maybe you have a plugin that adds extra margin/space? Could you try with a clean configuration, e.g. geany -c /tmp/tempconf?

@philiprbrenan
Copy link
Author

The problem only occurs in files with lots of lines - the largest line number is used to determine the width of this area as can be seen by starting with a window with one line in it and duplicating it 1000 times (very easy to do in Geany with Ctrl-D). In files that do have a large number of lines say 100,001 lines, then most lines will have at least one redundant space and if one tends to work at the top of the file most of the time there are 4 redundant spaces in this area.

@b4n
Copy link
Member

b4n commented Sep 28, 2016

The only way then would be to change the width of that margin depending on which lines are shown, which is likely to be annoying behavior when near a boundary (e.g. going up/down a line could shift the whole display to the left).

IMO this is at most a feature request, and I'd be tempted to close as WONTFIX. But maybe someone else would find this important enough to come up with a patch or a plugin that changes that (if it's a core patch, if would likely need a preference).

@elextr
Copy link
Member

elextr commented Sep 28, 2016

Scintilla does not allow variable margin widths, it is set for the entire buffer. If the space before the number is removed it will have to be put back somewhere else to keep alignment of the text.

Agree with @b4n this is not a sensible change.

@elextr elextr closed this as completed Sep 28, 2016
@codebrainz
Copy link
Member

The only way then would be to change the width of that margin depending on which lines are shown, which is likely to be annoying behavior when near a boundary

+1. Even when going from 9 to 10 lines or 999 to 1000 lines I find the automatic adjustment distracting, it would much more so if it happened while scrolling.

Moreover, in terse languages like Perl, I would imagine a 100,000 line file would be extremely rare (unless auto-generated or something).

@codebrainz
Copy link
Member

Scintilla does not allow variable margin widths, it is set for the entire buffer.

Actually scintilla doesn't handle any margin widths at all, leaving it entirely up to the application. Geany just follows the approach recommended in the Scintilla documentation and that taken by other (non-Scintilla) editors.

@vstepaniuk
Copy link

What about this:

screenshot from 2018-01-19 23-12

@codebrainz
Copy link
Member

@Stepaniuk I think it's from the underscore here. My guess would be that it was added to allow for using variable-width fonts or such.

@elextr
Copy link
Member

elextr commented Jan 20, 2018

I have the sidebar on the left and visible, then the space @Stepaniuk points to is the resize handle, it seems to only be "space" when the sidebar is right or hidden, so possibly its then the pane border as that edge of the edit pane is now on the outside of the pane?

@vstepaniuk
Copy link

screenshot from 2018-01-20 11-27-45

@elextr
Copy link
Member

elextr commented Jan 20, 2018

Clearly theme dependent
screenshot from 2018-01-20 19-33-15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants