From: Jakub Narebski <jnareb@gmail.com>
To: Junio C Hamano <junkio@cox.net>
Cc: Luben Tuikov <ltuikov@yahoo.com>, git@vger.kernel.org
Subject: Re: [PATCH] gitweb: tree view: eliminate redundant "blob"
Date: Sun, 1 Oct 2006 20:41:14 +0200 [thread overview]
Message-ID: <200610012041.15296.jnareb@gmail.com> (raw)
In-Reply-To: <7v64f9hku2.fsf@assigned-by-dhcp.cox.net>
On Wed, 27 Sep 2006, Junio C Hamano wrote:
> If the only objection is that it is harder to realize that the
> remaining one (the other one that did not get removed by this
> redundancy elimination) is clickable, maybe that is what needs
> to be fixed.
But that is plain impossible without compromising interface usability.
Let me explain.
In the case of tree view (directory listing) we have blobs (files and
symlinks) and trees (directories). To mark item unequivocably as link
it has to have default link color (blue) and default link decoration
(underline). That means that we cannot distinguish really well (at
least color) between tree and blob entries. I'd rather have redundant
"blob"/"tree" (self)links, clearly marked as links, and tree entries
using link decoration (blue, underlined) while blob entries have
default text decoration (black, no underline).
In the case of shortlog/log/history/heads/tags view, to clearly mark
subject/title of a commit or tag as link, we would have to use default
link decoration. Let's for a while abandon link-within-link, i.e. using
some of committags also in commit title (in shortlog/history view)...
But underlined text is harder to read, and blue underlined text even
more so (as for example it is hard to read italics, commonly used for
emphasis). I'd rather have additional "commit" link, clearly marked as
link, and leave subject as is, as hidden link, as a shortcut.
I think that redundancy in a visual interface (and not only visual, as
seen in the example of Perl programming language) is a good idea,
contrary to the redundancy in code or data (database).
--
Jakub Narebski
Poland
next prev parent reply other threads:[~2006-10-01 18:41 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-26 5:38 [PATCH] gitweb: tree view: eliminate redundant "blob" Luben Tuikov
2006-09-26 6:34 ` Junio C Hamano
2006-09-26 17:04 ` Luben Tuikov
2006-09-26 8:54 ` Jakub Narebski
2006-09-26 9:22 ` Jakub Narebski
2006-09-26 16:07 ` Petr Baudis
2006-09-26 16:24 ` Jakub Narebski
2006-09-26 20:33 ` Luben Tuikov
2006-09-27 2:40 ` Junio C Hamano
2006-10-01 18:49 ` Jakub Narebski
2006-09-26 20:14 ` Luben Tuikov
2006-09-26 20:31 ` Jakub Narebski
2006-09-26 21:32 ` Luben Tuikov
2006-09-26 22:24 ` Jakub Narebski
2006-09-26 22:30 ` Jakub Narebski
2006-09-27 6:42 ` Junio C Hamano
2006-10-01 18:41 ` Jakub Narebski [this message]
2006-10-01 18:56 ` Junio C Hamano
2006-10-01 19:27 ` Jakub Narebski
2006-10-02 7:15 ` Andreas Ericsson
2006-10-02 10:56 ` Jakub Narebski
2006-10-02 7:34 ` Junio C Hamano
2006-10-02 11:06 ` Jakub Narebski
2006-10-02 19:46 ` Luben Tuikov
2006-10-02 19:11 ` Luben Tuikov
2006-10-02 20:03 ` Jakub Narebski
2006-10-03 4:14 ` Junio C Hamano
2006-10-03 8:18 ` Jakub Narebski
2006-10-03 9:34 ` Junio C Hamano
2006-10-03 10:15 ` Jakub Narebski
2006-10-05 0:15 ` Luben Tuikov
2006-10-03 20:20 ` Luben Tuikov
2006-10-03 16:31 ` Linus Torvalds
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200610012041.15296.jnareb@gmail.com \
--to=jnareb@gmail.com \
--cc=git@vger.kernel.org \
--cc=junkio@cox.net \
--cc=ltuikov@yahoo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).