search

fork-dev / Tracker

Bug and issue tracker for Fork for Mac
510 stars 12 forks source link

[Feature Request] Commit message formatting #879

Open cakeray opened 4 years ago

cakeray commented 4 years ago

Hi All. Started using Fork a few weeks ago and I love it. One thing I do miss from my previous workflow is having commit messages formatted as you type.

I am a stickler for having commit messages formatted (I know a lot of people aren't) so they don't go beyond a certain column limit. Depending on which sites or tools you use for code reviews, etc., something like this could improve readability of commit messages. Seeing as there is already an option to display a page guide at a column limit in commit descriptions, I was wondering if there are plans to include a formatter for the commit messages. Maybe something that could introduce a newline once the column limit is reached and maybe respect tab sizes for bullet points?

Would love it if something like this could make it to a future release. For me, this lack of customization is the only flaw in an otherwise great git client.

DanPristupov commented 4 years ago

Preferences -> Show page guideline

then

Right click on commit description field -> Wrap Paragraph at Ruler

cakeray commented 4 years ago

Thanks for letting me know about this. I wasn't aware that such an option existed.

I tried this out and noticed that I have to select all the content in the description box and then manually wrap it using that option. Is there any way to leave this option "on" the entire time so the user doesn't have to do this every time?

cakeray commented 4 years ago

Also, I have noticed that when this option is used, all bullet points become a single large paragraph. Is there any way newlines can be maintained when formatting?

image

When the "Wrap paragraph at ruler" option is used, it becomes:

image

DanPristupov commented 4 years ago

Is there any way to leave this option "on" the entire time so the user doesn't have to do this every time?

No. I think this should be a manual action only.

Is there any way newlines can be maintained when formatting?

No. Fork merges consecutive lines but respects the paragraphs. This was a part of the original request: https://github.com/ForkIssues/Tracker/issues/17

cakeray commented 4 years ago

No. Fork merges consecutive lines but respects the paragraphs. This was a part of the original request: #17

Ah, I see. Looking through that ticket helped me understand how this works.

Are there any plans to add a hotkey to speed up the process, rather than having to use the drop down each time?