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)
committerGalen Charlton <galen.charlton@liblime.com>
Thu, 23 Oct 2008 20:41:18 +0000 (15:41 -0500)
commit143b757cfe0eed322bcc220070b9e683e16cb793
treeea86d1c0c6e0321ec2ccb99517e0453ab9396547
parentb909fb8061938b8eb9182d0a8ef7df0c7f89af42
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