LP#1507845: Improve search engine optimization Back in bug # 1414033 , we added rel="nofollow" attributes to many of the links that led to search results or call number browsing. This was the right idea, but a bit overzealous as it turns out. We actually want search engines to follow those links; we just don't want search engines to index them. Therefore, we can tweak the <meta> tag in the results and browse page headers to say "follow,noindex", and remove many of the inline rel="nofollow" attributes from the other pages. In addition, add a rel="canonical" link to the catalogue home page. Signed-off-by: Dan Scott <dscott@laurentian.ca> Signed-off-by: Ben Shum <ben@evergreener.net> Conflicts: Open-ILS/src/templates/opac/parts/record/subjects.tt2
LP #1406350 Mobile Device Navigation Issue Fix for Shelf Browser Currently, when a browser window is narrowed too much, whether in a mobile browser, or on a desktop or laptop machine, the "Previous Page" and "Next Page" navigation links disappear. The only solution that seems to solve this is by removing the "THEAD" element from the shelf browser table. This patch removes that tag, adds a new row to imitate a new border as existed before, and modifies the CSS in .../opac/ css/style.css.tt2 to address that difference. Signed-off-by: Adam Bowling <abowling@emeralddata.net> Signed-off-by: Yamil Suarez <yamil@yamil.com> Signed-off-by: Ben Shum <bshum@biblio.org>
LP#1414033: Add rel="nofollow" to search/browse links Mark links in the TPAC that robots should not attempt to crawl (such as search results, call number browsing, adding records to temporary lists, placing holds, logging in) using the rel="nofollow" convention, per https://support.google.com/webmasters/answer/96569?hl=en Signed-off-by: Dan Scott <dscott@laurentian.ca> Signed-off-by: Ben Shum <bshum@biblio.org>
LP#1268636 alt tags and cleanup continued Signed-off-by: Bill Erickson <berick@esilibrary.com> Signed-off-by: Dan Scott <dscott@laurentian.ca>
Correct small errors in new AC cover code In shelf browser, two problems: - missing '%]' after 'args=rec_attrs;' - cn.record_id should be cn.record.id Signed-off-by: Dan Wells <dbw2@calvin.edu> Signed-off-by: Ben Shum <bshum@biblio.org>
Teach TPAC and selfcheck jackets by record ID Teach TPAC and web-based selfcheck to display jackets by record ID. Not covered yet: KPAC (uses a non-1x1 placeholder at this time) and Acq (where we don't always have a bre.id). Signed-off-by: Jeff Godin <jgodin@tadl.org> Signed-off-by: Ben Shum <bshum@biblio.org>
TPAC: Render record titles and authors consistently Yamil Suarez noticed that the titles and authors in the call number browse results page were presented differently than the regular search results. This change removes some of the hard-coded <em> tags and "bold" classes, in favour of new "record_author" and "record_title" CSS classes. Now sites can simply change the CSS in one place to keep things consistent between the two search results pages. Signed-off-by: Dan Scott <dscott@laurentian.ca> Signed-off-by: Yamil Suarez <yamil@yamil.com> Signed-off-by: Ben Shum <bshum@biblio.org>
TPAC: Make call number browse record links retain search lib In addition to maintaining scope for the call number browse search results, we also need to maintain the scope for the links for the call number records in the results themselves (for example, the "retrieve this record" link, or the "search for this author" link). Signed-off-by: Dan Scott <dscott@laurentian.ca> Signed-off-by: Ben Shum <bshum@biblio.org>
TPAC - fix alignment of "You are now browsing..." in shelf browser The "You are now browsing..." message does not align to the center of the page. Changing the <div> to a <p> seems to properly align the text. Along the way, we should remove the extra <span> and <strong> (the strong is redundant with the css being inherited by color_4 class) and make the new single line message more i18n-friendly for future translations. Credit to Michael Peters for noticing the problem and thanks to Dan Scott, Lebbeous Fogle-Weekley, and Thomas Berezansky for help via IRC for the i18n-bits. Signed-off-by: Ben Shum <bshum@biblio.org> Signed-off-by: Michael Peters <mrpeters@library.in.gov> Signed-off-by: Dan Scott <dscott@laurentian.ca>
Fix TPAC shelf browser's previous link Line fix suggested by Thomas Berezansky. The cnoffset was strictly checking for values that start with a digit. But negative numbers start with a "-" instead. Once this check was changed, the previous link will continue to move further into negative offsets. Signed-off-by: Ben Shum <bshum@biblio.org> Signed-off-by: Dan Scott <dscott@laurentian.ca>
Remove /default/ from template paths The original goal of having a /default/ component to the template path was to support alternate skins. However, Template Toolkit supports alternate skins through a more elegant mechanism of providing template overlays via template path configuration. In other words, skin files will be differentiated by the location of their root path and not because they live in a different skin directory inside the main template root. Because of this, the /default/ component in the template path serves no purpose and generally just gets in the way. As a part of this, EGWeb.pm no longer inserts the skin name into the template file path. Signed-off-by: Bill Erickson <berick@esilibrary.com>