From mboxrd@z Thu Jan 1 00:00:00 1970 From: john at keeping.me.uk (John Keeping) Date: Thu, 24 Nov 2016 19:14:00 +0000 Subject: [PATCH v5 1/1] ui-shared: allow to split the repository link In-Reply-To: References: <1463067852-22123-1-git-send-email-petr.vorel@gmail.com> <20161124173220.nle2w5p2yk34pgqz@x230> Message-ID: <20161124191400.GE24063@john.keeping.me.uk> On Thu, Nov 24, 2016 at 06:42:32PM +0100, Jason A. Donenfeld wrote: > On Thu, Nov 24, 2016 at 6:32 PM, Petr Vorel wrote: > > Any idea how to cope with it? I thought to use this feature only if repo.name and repo.url > > are the same. > > One way might be to always use the `name` part and not the `url`, but > to ensure that any clickable links are actually useful. Namely, ensure > that any clickable links go to subtrees that contain other > repositories. This would also help collapse single-child directories, > like what we now do in tree view. > > @John - any opinions on this? I originally suggested using this only when repo.name and repo.url are the same and I still can't think of a good behaviour for when they differ. I guess if there is a common path prefix between repo.name and repo.url then we could implement this behaviour for that, but that feels like we're getting too far into edge cases. Basically, this feels like a good initial implementation that punts the complexity for us to deal with when/if someone has a use case. It also has the advantage of being easy to explain in documentation. > >> I realize that John asked for a config variable in v3. But I do wonder > >> if this really is so necessary. In what case would somebody _not_ want > >> this behavior? > > I don't think so, I'd also remove the config variable. > > I'll defer to John, then, to defend having a config variable. It looks like my original argument was essentially "some people might object to the change in behaviour", so I have no problem with just changing it and adding a config variable later if people scream.