D3665: graph: improve graph output by using Unicode characters

Sean Farley sean at farley.io
Thu Jun 14 19:02:37 EDT 2018


johnstiles (John Stiles) <phabricator at mercurial-scm.org> writes:

> johnstiles added a comment.
>
>
>   Hi there @indygreg and @yuja  -- are you happy with the diff as it currently stands? Do you think this needs any additional work?

Sorry for the late reply; was having email filtering issues (hopefully
fixed now).

Overall, I'm +0 on it because I think the "real" solution to this is to
templatize the graph glpyhs so any user could use whatever unicode they
desire. However, that's a tall order I wouldn't put on you so I'm fine
with putting this in and adding a note along the lines of, "the path to
get this in core is to get the graph log templated."

The only hiccup I keep having is the misalignment of graphs like these:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screen Shot 2018-06-14 at 4.00.50 PM.png
Type: image/png
Size: 6916 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180614/733c6ef6/attachment.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screen Shot 2018-06-14 at 4.00.57 PM.png
Type: image/png
Size: 7420 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180614/733c6ef6/attachment-0001.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screen Shot 2018-06-14 at 4.01.25 PM.png
Type: image/png
Size: 6642 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180614/733c6ef6/attachment-0002.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screen Shot 2018-06-14 at 4.01.17 PM.png
Type: image/png
Size: 9301 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180614/733c6ef6/attachment-0003.png>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Screen Shot 2018-06-14 at 4.01.03 PM.png
Type: image/png
Size: 6837 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180614/733c6ef6/attachment-0004.png>
-------------- next part --------------

Do you think there's any way to fix those? I suspect the answer is "no"
unless someone wants to write their own font?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180614/733c6ef6/attachment.sig>


More information about the Mercurial-devel mailing list