From: Stephen Cameron <stephen@cubedesigners.com>
Date: Wed, 19 Dec 2018 09:59:19 +0000 (+0100)
Subject: Suppress bookmarks option in print/download modal when bookmarks are disabled. WIP... 
X-Git-Url: http://git.cubedesigners.com/?a=commitdiff_plain;h=905bcbe372c44905d4ef302a18c98b7f305915ca;p=fluidbook-html5.git

Suppress bookmarks option in print/download modal when bookmarks are disabled. WIP #1806 @0.25
---

diff --git a/js/libs/fluidbook/fluidbook.print.js b/js/libs/fluidbook/fluidbook.print.js
index e81d3f0b..2bbed8ee 100644
--- a/js/libs/fluidbook/fluidbook.print.js
+++ b/js/libs/fluidbook/fluidbook.print.js
@@ -144,18 +144,20 @@ FluidbookPrint.prototype = {
         view += '</div>'; // .print-option
 
         // Bookmarks
-        var hasBookmarks = this.fluidbook.bookmarks.hasBookmarkedPages(),
-        bookmarksDisabled = hasBookmarks ? '' : 'disabled';
-
-        view += '<div class="print-option bookmarks-option '+ bookmarksDisabled +'">';
-        view += '<label for="bookmarkedPages">';
-        view += '<div class="doubleThumb bookmarks">';
-        view += this.fluidbook.bookmarks.getPrintPreview();
-        view += '</div>'; // .doubleThumb
-        view += '</label>';
-        view += '<input type="radio" name="pageChoice" value="bookmarks" id="bookmarkedPages" '+ bookmarksDisabled +'>'
-        view += '<label for="bookmarkedPages" class="print-label-text">' + __('bookmarks') + '</label>';
-        view += '</div>'; // .print-option
+        if (this.fluidbook.datas.bookmark) {
+          var hasBookmarks = this.fluidbook.bookmarks.hasBookmarkedPages(),
+          bookmarksDisabled = hasBookmarks ? '' : 'disabled';
+
+          view += '<div class="print-option bookmarks-option '+ bookmarksDisabled +'">';
+          view += '<label for="bookmarkedPages">';
+          view += '<div class="doubleThumb bookmarks">';
+          view += this.fluidbook.bookmarks.getPrintPreview();
+          view += '</div>'; // .doubleThumb
+          view += '</label>';
+          view += '<input type="radio" name="pageChoice" value="bookmarks" id="bookmarkedPages" '+ bookmarksDisabled +'>'
+          view += '<label for="bookmarkedPages" class="print-label-text">' + __('bookmarks') + '</label>';
+          view += '</div>'; // .print-option
+        }
 
         view += '<div class="print-option"></div>'; // Extra empty div to ensure flexbox space-between works as expected