[open-ils-commits] ***SPAM*** [GIT] Evergreen ILS branch master updated. eda58b7612de8c7438d545b6b97693b751d46e4b

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, master has been updated
       via  eda58b7612de8c7438d545b6b97693b751d46e4b (commit)
      from  d445150a1adb9b696dbf3531eca9d31d65224d5f (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit eda58b7612de8c7438d545b6b97693b751d46e4b
Author: Dan Scott <dscott at laurentian.ca>
Date:   Mon Jan 12 13:13:03 2015 -0500

    LP#1409844: Towards more meaningful catalogue <title> elements
    
    Currently the catalogue <title> elements are formed by concatenating the page
    title to "Catalog - ". This means that on browsers with more than a few tabs
    open, users will only see "Catalog -", which doesn't help them navigate their
    tabs. Also, in search results in general search engines, "Catalog - " is
    expected to be a significant word as it appears first in the title of the page.
    
    Instead, let's remove the "Catalog - " prefix entirely and add "- <library
    name>" as a suffix to the page title. That way, a user will see the page title
    in bookmarks, search results, and other contexts with the most important
    information up front (whether that's the title of the work in record details,
    or the fact that it's a search for some string), followed by the name of the
    library to give it a specific context.
    
    In addition, use the name of the library in the OpenSearch definition to be
    more meaningful than the current hardcoded "Evergreen OpenSearch" (many users
    will not even know what Evergreen is, if they stumble across the little "Add
    search engine" widget at all).
    
    And finally, use the HTML and URI filters appropriately to avoid any possible
    damage :)
    
    Signed-off-by: Dan Scott <dscott at laurentian.ca>
    Signed-off-by: Ben Shum <bshum at biblio.org>

diff --git a/Open-ILS/src/templates/opac/parts/base.tt2 b/Open-ILS/src/templates/opac/parts/base.tt2
index 03e950d..4cd408f 100644
--- a/Open-ILS/src/templates/opac/parts/base.tt2
+++ b/Open-ILS/src/templates/opac/parts/base.tt2
@@ -11,9 +11,12 @@
         <meta name = "viewport" content = "initial-scale = 1.0">
         <link rel="stylesheet" type="text/css" href="[% ctx.media_prefix %]/css/skin/default/opac/semiauto.css" />
         <link rel="stylesheet" type="text/css" href="[% ctx.opac_root %]/css/style.css" />
-        <title>[% l('Catalog - [_1]', ctx.page_title) %]</title>
+        [%- libname = ctx.get_aou(ctx.search_ou).name;
+            libname = libname | html;
+            libsname = ctx.get_aou(ctx.search_ou).shortname; %]
+        <title>[% l('[_1] - [_2]', ctx.page_title, libname) %]</title>
         <link rel="unapi-server" type="application/xml" title="unAPI" href="/opac/extras/unapi" />
-        <link type="application/opensearchdescription+xml" rel='search' title="[% l('Evergreen OpenSearch') %]" href="/opac/extras/opensearch/1.1/[% ctx.get_aou(ctx.search_ou).shortname %]/-/osd.xml" />
+        <link type="application/opensearchdescription+xml" rel='search' title="[% l('[_1] OpenSearch', libname) %]" href="/opac/extras/opensearch/1.1/[% libsname | uri %]/-/osd.xml" />
         [% IF want_dojo %]
         <style type="text/css">
             @import "[% ctx.media_prefix %]/js/dojo/dijit/themes/tundra/tundra.css";

-----------------------------------------------------------------------

Summary of changes:
 Open-ILS/src/templates/opac/parts/base.tt2 |    7 +++++--
 1 files changed, 5 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
Evergreen ILS