git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jakub Narebski <jnareb@gmail.com>
To: git@vger.kernel.org
Subject: Re: [PATCH 2/2] gitweb: Show trailing slash when listing tree entry in tree listing
Date: Tue, 10 Oct 2006 11:15:22 +0200	[thread overview]
Message-ID: <egfo99$lg6$2@sea.gmane.org> (raw)
In-Reply-To: 20061010053841.42852.qmail@web31815.mail.mud.yahoo.com

Luben Tuikov wrote:

>> It probably is wise to resurrect those "redundant" links.
> 
> If someone does this, can they also remove the now "other"
> redundant link? (the link at the pathname itself) A simple
> code analyzer would show the duplicate code in gitweb.

Easy, easy now.


I'd rather add some more "hidden" links, but for each hidden
link (which are convenience only, to have larger are to click,
or to have closer area to click) I'd like to have clearly marked
link (marked as a link, i.e. using default link style; and with link text
denoting _kind_ of link) which leads to the same contents. 

For example on project list page I would made also project description
(and not only project name) clickable, leading tp project summary.
Making project name direct link wouldn't work for sites like kernel.org
with long (hierarchical) project names like
  linux/kernel/git/wim/linux-2.6-watchdog-experimental.git
And for other sites project name is/can be bit on the short side.

But we agreed (I guess) to disagree on the whole redundancy in user
interface issue (although I agree on the issue of reducing clutter).
BTW. we can reduce redundancy in the code without need for removing
"alternate entry points" in interface, I think.
-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git

  parent reply	other threads:[~2006-10-10  9:16 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-05 19:22 [PATCH] gitweb: Do not print "log" and "shortlog" redundantly in commit view Luben Tuikov
2006-10-06  7:44 ` Jakub Narebski
2006-10-06 16:35   ` Petr Baudis
2006-10-06 22:16   ` Luben Tuikov
2006-10-07 13:24     ` Petr Baudis
2006-10-07 14:05       ` Jakub Narebski
2006-10-07 14:17         ` Petr Baudis
2006-10-07 14:10       ` [PATCH 1/2] gitweb: Show snapshot links for tree entries in tree listing Petr Baudis
2006-10-07 18:37         ` Luben Tuikov
2006-10-07 18:41           ` Petr Baudis
2006-10-07 18:52             ` Luben Tuikov
2006-10-07 19:15               ` Petr Baudis
2006-10-07 22:31                 ` Junio C Hamano
2006-10-08  1:04                 ` Luben Tuikov
2006-10-07 14:10       ` [PATCH 2/2] gitweb: Show trailing slash when listing tree entry " Petr Baudis
2006-10-07 18:44         ` Luben Tuikov
2006-10-07 19:06           ` Jakub Narebski
2006-10-07 19:12             ` Petr Baudis
2006-10-07 20:38               ` Jakub Narebski
2006-10-07 21:15                 ` Petr Baudis
2006-10-07 21:27                   ` Jakub Narebski
2006-10-09 20:55               ` Petr Baudis
2006-10-09 22:52                 ` Junio C Hamano
2006-10-10  5:10                   ` Junio C Hamano
2006-10-10  5:38                     ` Luben Tuikov
2006-10-10  5:46                       ` Junio C Hamano
2006-10-10  6:34                         ` Luben Tuikov
2006-10-10  5:46                       ` Jeff King
2006-10-10  6:41                         ` Luben Tuikov
2006-10-10  6:58                           ` Jeff King
2006-10-10  9:15                       ` Jakub Narebski [this message]
2006-10-10 19:19                         ` Luben Tuikov
2006-10-10 19:57                           ` Junio C Hamano
2006-10-10 20:31                             ` Jakub Narebski
2006-10-10 21:02                               ` Luben Tuikov
2006-10-10 21:13                                 ` Jakub Narebski
2006-10-10 22:18                                   ` Luben Tuikov
2006-10-10 20:52                             ` Luben Tuikov
2006-10-10 21:00                               ` Jakub Narebski
2006-10-10 22:14                                 ` Luben Tuikov
2006-10-10 22:40                                   ` Jakub Narebski
2006-10-10 20:29                           ` Jakub Narebski
2006-10-11 15:35                           ` Andreas Ericsson
2006-10-10  6:21                 ` Luben Tuikov
2006-10-10  7:05                   ` Jeff King
2006-10-10  8:07                     ` Andreas Ericsson
2006-10-10 13:14                       ` Josef Weidendorfer
2006-10-10 18:23                         ` Luben Tuikov
2006-10-10 18:52                           ` Josef Weidendorfer
2006-10-10 18:50                         ` Luben Tuikov
2006-10-10  8:28                     ` Junio C Hamano
2006-10-07 21:31           ` A Large Angry SCM
2006-10-07 22:34           ` Junio C Hamano
2006-10-08  1:16             ` Luben Tuikov

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='egfo99$lg6$2@sea.gmane.org' \
    --to=jnareb@gmail.com \
    --cc=git@vger.kernel.org \
    /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).