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

Prevent text superposing git blame #684

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

cesarmontalverne
Copy link
Contributor

@cesarmontalverne cesarmontalverne commented Jan 28, 2024

Change to prevent git blame gutter from superposing text in smaller screens. There is no change for screens larger than 1000px
Before:
image

After:
image

Before and After for larger screens:
image

@cesarmontalverne cesarmontalverne changed the title Hide git blame gutter not to superpose text for smaller screens Prevent text superposing git blame - 7 lines css change Jan 28, 2024
@cesarmontalverne cesarmontalverne changed the title Prevent text superposing git blame - 7 lines css change Prevent text superposing git blame Jan 28, 2024
@Vinzent03
Copy link
Collaborator

I don't have that issue when shrinking the window size. Do you use any theme or custom css snippets?

@cesarmontalverne
Copy link
Contributor Author

cesarmontalverne commented Jan 28, 2024

I do use a theme but I got that even when I didn’t. Maybe it’s my OS?

I love this plugin by the way - thanks for having made it!

@Vinzent03
Copy link
Collaborator

The real issue here is the minimal theme, which somehow occupies the gutter, which the line authoring uses. It happens in full screen as well, with readable line length disabled.

I will merge this at a width of 800px, but just because it's a general fix to hide it on small widths.

styles.css Outdated Show resolved Hide resolved
@Vinzent03 Vinzent03 merged commit 6a89424 into denolehov:master Mar 14, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants