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

Indent guide misalignment when using line or caret padding #1185

Closed
anlutro opened this issue Apr 26, 2016 · 3 comments
Closed

Indent guide misalignment when using line or caret padding #1185

anlutro opened this issue Apr 26, 2016 · 3 comments

Comments

@anlutro
Copy link

anlutro commented Apr 26, 2016

Using Sublime Text 3, build 3111, on Debian 8 installed using the .deb package.

If you use the indent guide with any amount of line_padding_top, it seems like the indent guide doesn't match up with the line highlight as well as the caret.

Notice how the indent guide line extends a few pixels below the highlighted line. If I set line_padding_bottom and line_padding_top both to 0, this is no longer an issue.

EDIT: Ignore the text below, see comments

If I set line_padding_bottom and line_padding_top both to 0, I see another problem: Because I have caret_extra_top and caret_extra_bottom both set to 1, the indent guide is still 1px "below" the highlighted line.

The last image is obviously more subtle than the first... but if I move the highlighted line to the next line, you can see that unlike the previous line there is no padding between the start of the highlighted line and the indent guide. If the indent guide were 1px higher up, the padding would be identical on the two lines.

@FichteFoll
Copy link
Collaborator

I'm trying really hard but I just can't find the difference between the last two pictures...

@anlutro
Copy link
Author

anlutro commented Apr 27, 2016

Upon checking more thoroughly I think the latter issue I brought up can be ignored. The length of the indent guide line seems to vary from line to line so it looked a bit more significant than it actually is in the example I brought up. I'll edit the original issue to reflect this.

@anlutro
Copy link
Author

anlutro commented Nov 1, 2017

Fixed in 3153!

@anlutro anlutro closed this as completed Nov 1, 2017
@wbond wbond added the R: fixed label Nov 1, 2017
@wbond wbond added this to the Build 3153 milestone Nov 1, 2017
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

3 participants