@pmatilai commented on this pull request.


> @@ -71,57 +71,57 @@ to perform useful operations. The current list is
        %getconfdir     expand to rpm "home" directory (typically /usr/lib/rpm)
        %dnl            discard to next line (without expanding)
        %verbose        expand to 1 if rpm is in verbose mode, 0 if not
-       %{verbose:...}  expand to ... if rpm is in verbose mode, the
+       %{verbose:…}    expand to … if rpm is in verbose mode, the

I don't think the use of unicode ellipsis ("…") is an improvement at all. It 
turns a plain and simple thing into something you have to look up how to type, 
and readability becomes very renderer-dependent. In this comment box, it's 
rendered reasonably, but in the above line and my terminal the three dots are 
so close together it's just unreadable with tired eyes.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2251#pullrequestreview-1163291403
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2251/review/1163291...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to