Unravelling links in masthead. Regularize logic and improve readability.
authorJoe Atzberger <joe.atzberger@liblime.com>
Fri, 19 Sep 2008 21:24:22 +0000 (16:24 -0500)
committerHenri-Damien LAURENT <henridamien.laurent@biblibre.com>
Wed, 12 Nov 2008 14:12:34 +0000 (15:12 +0100)
commit2a12fec0f8ebfc179ecb5687a7347250cf47bf1d
tree410860a0524410fc91b22664b0c2f8d9296459f7
parent17e956aeedbd5964dd6da4c04c1758bd9784959f
Unravelling links in masthead. Regularize logic and improve readability.

Link to Tag Cloud displayed on Adv. Search page even when TagsEnabled was OFF.
This was due to unnecessary duplicative code for Adv. Search vs. other pages.
Note: with this revision, the link to "Advanced Search" will display on all pages,
even on the Advanced Search page itself.  This is in fact desirable, rather than
the set of links collapsing and changing position depending on page being viewed.
If the old premise were to be followed (that Adv. Search link should not display on its
own page), then the same interface sensibility should apply to ALL links in the
masthead: i.e., Tag Cloud link should not display on its own page, same for Browse
by Subject, Most Popular, etc.  I would argue even this is undesirable.  At most,
give the links in moresearches some IDs and let each page unlink/restyle the
"current" link with jquery.  IMHO, masthead itself should not be engage in such trifling
interface polishes.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
koha-tmpl/opac-tmpl/prog/en/includes/masthead.inc