While not often used, tag inheritance can be modified with a few different options (e.g. maxdepth). These options are shown in the taginfo display, but not the list-tag-inheritance display. This patch adds basic indicators to the latter.
Users can be easily confused when these options are in use, so better to make things more visible.
I'm not set on the exact display. I tried a few variations. The 4 character flag display is concise, but doesn't look all that great without alignment. Also, most inheritance lines have no options, so the dots seem repetitive and noisy.
Metadata Update from @mikem: - Pull-request tagged with: usability
:thumbsup:
Let me show an example so that folks can see that this looks like
Original (without these changes)
$ koji list-tag-inheritance tag_y tag_y (198) └─tag_a (53)
With current changes:
$ koji list-tag-inheritance tag_y tag_y (198) └─tag_a (53) M...
(indicates the presence of maxdepth option on the inheritance line)
Another example. No inheritance options in the chain.
$ koji list-tag-inheritance tag_a tag_a (53) ├─tag_b (52) .... └─tag_c (193) .... └─tag_d (194) ....
I'm not loving the way this looks, but I've so far been unable to come up with a better option that is still compact.
should priority be shown?
That would make the display even busier, and the ordering already reflects relative priority.
I guess it could be useful, but I don't think it needs to be part of this PR
Does this look better?
$ koji list-tag-inheritance tag_a_foo tag_a_foo (223) .... └─tag_a (53) .... ├─tag_b (52) .... └─tag_c (193) .... └─tag_d (194)
$ koji list-tag-inheritance tag_y tag_y (198) M... └─tag_a (53)
Maybe be, it is better. But both approaches work for me.
1 new commit added
move flags to front
Filed #1625 to track this
@julian8628 @breilly
Metadata Update from @tkopecek: - Pull-request tagged with: testing-ready
I prefer having flags at the start as well.
Commit 8e548962 fixes this pull-request
Pull-Request has been merged by tkopecek
While not often used, tag inheritance can be modified with a few different options (e.g. maxdepth). These options are shown in the taginfo display, but not the list-tag-inheritance display. This patch adds basic indicators to the latter.