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

Multiple spaces get lost in many UIs #26

Closed
JayBazuzi opened this issue Jul 1, 2021 · 10 comments · Fixed by #39
Closed

Multiple spaces get lost in many UIs #26

JayBazuzi opened this issue Jul 1, 2021 · 10 comments · Fixed by #39
Labels
enhancement New feature or request

Comments

@JayBazuzi
Copy link
Collaborator

We currently use three spaces for safer commits (r Extract Method) so that they align well with unsafe commits (R!! Cleaned up the core logic) but that only works with fixed-width display. Many UIs merge those spaces, like this in GitHub:

image

@JayBazuzi
Copy link
Collaborator Author

On one project we're using r - . Maybe something like that would work?

Maybe r.. ?

@arlobelshee
Copy link
Contributor

arlobelshee commented Jul 2, 2021

I personally prefer whitespace over .. as it makes the !! and ** stand out. Your proposal for - meets that well (I think). An example would be:

r - lorem ipsum
F - lorem ipsum
t - lorem ipsum
r - lorem ipsum
F!! lorem ipsum
r - lorem ipsum
R - lorem ipsum
B** lorem ipsum
F!! lorem ipsum
d - lorem ipsum

@arlobelshee
Copy link
Contributor

arlobelshee commented Jul 2, 2021

I'd love to hear thoughts from @isidore, @mscottford, or others on this proposal.

@JayBazuzi
Copy link
Collaborator Author

Just for comparison:

r.. lorem ipsum
F.. lorem ipsum
t.. lorem ipsum
r.. lorem ipsum
F!! lorem ipsum
r.. lorem ipsum
R.. lorem ipsum
B** lorem ipsum
F!! lorem ipsum
d.. lorem ipsum

@JayBazuzi
Copy link
Collaborator Author

And one more:

r__ lorem ipsum
F__ lorem ipsum
t__ lorem ipsum
r__ lorem ipsum
F!! lorem ipsum
r__ lorem ipsum
R__ lorem ipsum
B** lorem ipsum
F!! lorem ipsum
d__ lorem ipsum

@mscottford
Copy link
Contributor

My initial reaction is to wonder about the importance of the extra spaces. It appears that they are primarily needed to help the eye notice when !! is used. But I wonder if there might be another approach that could achieve the same goal.

!F! and .r. might be a good alternative. Because then instead of the .. or __ being used as padding characters, they are being used as delimeters. And then the delimiter conveys meaning in addition to the single character that it wraps.

.r. lorem ipsum
.F. lorem ipsum
.t. lorem ipsum
.r. lorem ipsum
!F! lorem ipsum
.r. lorem ipsum
.R. lorem ipsum
*B* lorem ipsum
!F! lorem ipsum
.d. lorem ipsum

I'm curious about y'all's thoughts on that.

@arlobelshee
Copy link
Contributor

Hm. I'm losing glance-ability with any of the options that use periods. My number 1 priority with this encoding is for someone to quickly glance down the column and, without reading, see the high-risk commits. I really want ** and !! to stand out. I'd like R to stand out some from r, but that is less important.

The underscores work a little better than periods for me at a glace, though still not as good as spaces (even with the hyphen).

@arlobelshee
Copy link
Contributor

!F! and .r. might be a good alternative. Because then instead of the .. or __ being used as padding characters, they are being used as delimeters. And then the delimiter conveys meaning in addition to the single character that it wraps.

I'm curious about y'all's thoughts on that.

What advantage do you see to having these as delimiters? Personally I don't get a psychological difference between delimiter and trailing marker. What are you noticing that I miss?

@mscottford
Copy link
Contributor

My initial thought along those lines was that if there were start and stop delimiters, instead of fixed width/padding, then the mechanism could employ longer terms instead of just single characters, and that this might increase understanding amongst other teams. Perhaps something similar to Gitmoji, but instead of relying on the emoji names, aliases are created.

I trimmed that back to the proposal that I sent in because I suspected that would be more drastic a change than was requested. That original idea looked something like this:

:safe: :refactor: lorem ipsum
:validated: :feature: lorem ipsum
:safe: :test: lorem ipsum
:safe: :refactor: lorem ipsum
:risky: :feature: lorem ipsum
:safe: :refactor: lorem ipsum
:risky: :refactor: lorem ipsum
:broken: :bugfix: lorem ipsum
:risky: :feature: lorem ipsum
:safe: :docs: lorem ipsum

Which I suspected would not meet the goals that you all have. However, this approach allows someone to very quickly scan the list for :safe:, :validated:, :risky: or :broken: by placing that detail first.

I'm noticing that this preference of having a start and stop character stems from me not being a very big fan of meaningful whitespace. In addition to GitHub not rendering the extra spaces, I notice that I often forget to actually type them out. :) It's completely possible that this approach is incompatible with the goals of this effort. :)

I'm also noticing that the suggestion that I've made above is influenced by a desire to avoid a single value having two different meanings (f and F** communicates both risk and change-type) and a desire to avoid the introduction of more initialisms since those tend to lead to "what does that mean" questions.

@arlobelshee
Copy link
Contributor

I'm going to close this with the adoption of - as the separator instead of .

I see the value of your proposal, Scott, but yeah, I don't find it to meet the goals of this notation. I'm explicitly looking for a columnar format, as that makes vertical scanning easy. Gitmoji would account for that, as long as the tooling supported it (and there was decent auto-complete when writing the commit message). I'd take your proposal over mine if those were both true!

JayBazuzi added a commit that referenced this issue Apr 23, 2022
jonreid added a commit to jonreid/GitCommitMessageTemplate that referenced this issue May 29, 2022
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.

3 participants