Thread (15 messages) 15 messages, 3 authors, 2016-06-15

Re: [PATCH] gitweb: allow extra breadcrumbs to prefix the trail

flat view

From: Tony Finch <dot@dotat.at>
Date: 2016-06-15 22:58:00

Jakub Narębski [off-list ref] wrote:
It would be better to improve documentation, than follow current bad
practice... ;-P
The v2 patch does just that :-)
Perhaps even make ( [ $home_link_str, $home_link ] ) to be default
value for @extra_breadcrumbs, making new feature generalization
of $home_link*, similarly to how it was done for $stylesheet -> @stylesheets
transition.
I don't think that's a win. There's a lot of existing gitweb.conf out
there which sets $home_link_str, so the code would have to either print
the last element of @extra_breadcrumbs or the $home_link variables
depending on whether the variables were modified. And the documentation
would have to explain this complicated arrangement.

Tony.
-- 
f.anthony.n.finch  [off-list ref]  http://dotat.at/
Forties, Cromarty: East, veering southeast, 4 or 5, occasionally 6 at first.
Rough, becoming slight or moderate. Showers, rain at first. Moderate or good,
occasionally poor at first.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help