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

templates: use "+" instead of "#" to denote immutable node in ascii graph #3542

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

yuja
Copy link
Contributor

@yuja yuja commented Apr 19, 2024

Maybe it's personal preference, but the hash sign looks bigger compared to the normal "o" nodes, and is slanted. This makes immutable commits stand out too much. I think "+" is closer to the diamond character used in the unicode template.

Checklist

If applicable:

  • I have updated CHANGELOG.md
  • I have updated the documentation (README.md, docs/, demos/)
  • I have updated the config schema (cli/src/config-schema.json)
  • I have added tests to cover my changes

…raph

Maybe it's personal preference, but the hash sign looks bigger compared to
the normal "o" nodes, and is slanted. This makes immutable commits stand out
too much. I think "+" is closer to the diamond character used in the unicode
template.
@martinvonz
Copy link
Member

@algmyr, @thoughtpolice

@algmyr
Copy link
Contributor

algmyr commented Apr 19, 2024

I'm fine with whatever, this was discussed a bit in the discord. I personally like + as well, but there was the worry that it looks like part of the graph. That might not be much of an issue with the color being different though.

@algmyr
Copy link
Contributor

algmyr commented Apr 19, 2024

I think as one of the (presumably few) users of the ascii theme, you have a big say in the symbols used. :)

Copy link
Contributor

@PhilipMetzger PhilipMetzger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense as another user on the not yet modern Windows terminal.

@necauqua
Copy link
Contributor

necauqua commented Apr 19, 2024

Pretty sure I was one of the people suggesting #, but I don't use ascii ¯\_(ツ)_/¯

+ is actually fine, yeah

@ilyagr
Copy link
Contributor

ilyagr commented Apr 19, 2024

(Non-blocking; I think I missed the Discord discussion)

Other possible symbols that come to my mind are % and =.

My mild preference would be %, if it's not too large for you. It kinda also looks like a diamond or a square (depending on the font, from a distance).

We might want to use + and - for diffing graphs for something like jj op diff. However, who knows if that idea ever materializes in that form and + sounds fine to me until then.

@yuja
Copy link
Contributor Author

yuja commented Apr 20, 2024

Other possible symbols that come to my mind are % and =.

To me, % doesn't look like a normal graph node symbol. Perhaps, I like + because it's symmetric in both horizontal and vertical directions. * was another option, but I felt + is easier to distinguish from o.

@yuja yuja merged commit cce1b9f into jj-vcs:main Apr 20, 2024
16 checks passed
@yuja yuja deleted the push-uuzpxplxkzps branch April 20, 2024 04:31
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.

7 participants