From eda58b7612de8c7438d545b6b97693b751d46e4b Mon Sep 17 00:00:00 2001 From: Dan Scott Date: Mon, 12 Jan 2015 13:13:03 -0500 Subject: [PATCH] LP#1409844: Towards more meaningful catalogue 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@laurentian.ca> Signed-off-by: Ben Shum <bshum@biblio.org> --- Open-ILS/src/templates/opac/parts/base.tt2 | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/Open-ILS/src/templates/opac/parts/base.tt2 b/Open-ILS/src/templates/opac/parts/base.tt2 index 03e950d439..4cd408ff5d 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) %] + [%- libname = ctx.get_aou(ctx.search_ou).name; + libname = libname | html; + libsname = ctx.get_aou(ctx.search_ou).shortname; %] + [% l('[_1] - [_2]', ctx.page_title, libname) %] - + [% IF want_dojo %]