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

Syntax highlighting with Verbose #1

Open
aubreypwd opened this issue Nov 11, 2024 · 3 comments · May be fixed by #4
Open

Syntax highlighting with Verbose #1

aubreypwd opened this issue Nov 11, 2024 · 3 comments · May be fixed by #4
Labels
enhancement New feature or request

Comments

@aubreypwd
Copy link

CleanShot 2024-11-11 at 12 42 13@2x

I was initially confused about what the dark vs. grey meant, I assume it's a 72 width counter? I think that could be made more obvious if so.

But my request here is to request syntax highlighting. I have verbose mode on when committing so I can see the changes briefly in the commit message, it is just white for now.

Great app, excited to see it improve!

@kra-mo kra-mo added the enhancement New feature or request label Nov 24, 2024
@kra-mo
Copy link
Owner

kra-mo commented Nov 24, 2024

 I assume it's a 72 width counter?

It is indeed, how do you think it could be made more obvious?

@kra-mo
Copy link
Owner

kra-mo commented Nov 24, 2024

For now, I added a description to the warning you get if you go over 50 characters with your commit message. Could add a warning for the description too, but I feel like this should be enough.

Screenshot 2024-11-24 at 20 23 53

@aubreypwd
Copy link
Author

It is indeed, how do you think it could be made more obvious?

At first, honestly, I thought it looked broken it took me a minute to realize that was probably the 72 limit. Maybe a harder line? But the message below is helpful.

@kra-mo kra-mo linked a pull request Dec 14, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants