From 5e223e2d08b52d3564ca4cab21250de0d1bf8715 Mon Sep 17 00:00:00 2001
From: Dan Wells <dbw2@calvin.edu>
Date: Mon, 25 Feb 2013 11:39:00 -0500
Subject: [PATCH] Tweak Google Book Preview Display

This commit makes three changes to the Google Book preview link:

1) Move the button to the same spot on the page as the viewer.
This clarifies the relationship between the button and the viewer,
and makes it far less likely that the viewer will load off the
screen.

2) Make the button a toggle for turning the viewer on and off.

3) In conjunction with (2), add some additional text to the button
link to follow the toggling action.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
---
 .../src/templates/opac/parts/ac_google_books.tt2   | 28 ++++++++++++++++++----
 .../tpac_google_books_preview.txt                  |  6 ++---
 2 files changed, 27 insertions(+), 7 deletions(-)

diff --git a/Open-ILS/src/templates/opac/parts/ac_google_books.tt2 b/Open-ILS/src/templates/opac/parts/ac_google_books.tt2
index 8243fe0738..e8119bf2f6 100644
--- a/Open-ILS/src/templates/opac/parts/ac_google_books.tt2
+++ b/Open-ILS/src/templates/opac/parts/ac_google_books.tt2
@@ -1,5 +1,6 @@
 <script type="text/javascript">
 var GBisbns = Array();
+var GBPreviewShowing = false;
 
 /**
  *
@@ -32,11 +33,13 @@ function GBPreviewCallback(GBPBookInfo) {
     GBPBadge.src = 'https://www.google.com/intl/[% ctx.locale.substr(0,2) %]/googlebooks/images/gbs_preview_button1.gif';
     GBPBadge.title = dojo.byId('rdetail_title').innerHTML;
     GBPBadge.style.border = 0;
-    GBPBadge.style.margin = '0.5em 0 0 0';
+    GBPBadge.style.verticalAlign = 'middle';
     GBPBadgelink = document.createElement('a');
-    GBPBadgelink.href = 'javascript:GBDisplayPreview();';
+    GBPBadgelink.id = 'gbptoggle';
     GBPBadgelink.appendChild( GBPBadge );
-    dojo.byId('rdetail_title_div').appendChild( GBPBadgelink );
+    GBPBadgelink.href = 'javascript:GBDisplayPreview();';
+    GBPBadgelink.appendChild( document.createTextNode(' [% l('Show Preview') %]') );
+    dojo.byId('canvas_main').insertBefore(GBPBadgelink, dojo.byId('rdetail_record_details'));
   }
 }
 
@@ -64,7 +67,24 @@ function GBPViewerLoadCallback() {
   var GBPViewer = new google.books.DefaultViewer(dojo.byId('rdetail_preview_div'));
   GBPViewer.load('ISBN:' + GBisbns[0]);
   GBPViewer.resize();
-  dojo.byId('gbpbadge').style.display = 'none';
+  var GBPBadgelink = dojo.byId('gbptoggle');
+  GBPBadgelink.href = 'javascript:GBShowHidePreview();';
+  GBShowHidePreview(GBPBadgelink);
+}
+
+function GBShowHidePreview(toggle) {
+  if (!toggle) {
+    toggle = dojo.byId('gbptoggle');
+  }
+  if (!GBPreviewShowing) {
+    dojo.byId('rdetail_preview_div').style.display = 'inherit';
+    toggle.lastChild.nodeValue = ' [% l('Hide Preview') %]';
+    GBPreviewShowing = true;
+  } else {
+    dojo.byId('rdetail_preview_div').style.display = 'none';
+    toggle.lastChild.nodeValue = ' [% l('Show Preview') %]';
+    GBPreviewShowing = false;
+  }
 }
 
 dojo.addOnLoad(function() {
diff --git a/docs/RELEASE_NOTES_NEXT/tpac_google_books_preview.txt b/docs/RELEASE_NOTES_NEXT/tpac_google_books_preview.txt
index c56582572c..d5ec36aaf7 100644
--- a/docs/RELEASE_NOTES_NEXT/tpac_google_books_preview.txt
+++ b/docs/RELEASE_NOTES_NEXT/tpac_google_books_preview.txt
@@ -4,9 +4,9 @@ TPAC: Google Books preview
 Setting `ctx.google_books_preview` to `1` in the TPAC `config.tt2`
 configuration file will cause the TPAC to check to see, as part of the record
 details view, if Google Books has an online preview available. If it does,
-then a preview button will be displayed in the book cover image location.
-If the user then clicks the preview button, the preview will load below the
-title of the book in the record details page.
+then a preview button will be displayed below the list of copies. If the user
+then clicks the preview button, the preview will load below the button. At that
+point, the button will then will act as a toggle to turn the viewer on and off.
 
 By default, this functionality is disabled to protect the privacy of users
 who might not want to share their browsing behaviour with Google.
-- 
2.11.0