Factorize a little all fancybox stuff. re #73

index.php

@@ -81,20 +81,9 @@
             });
             
 <?php if (getCurrentOption ('use_fancyapps') == 1) { ?>
-            $(".fancycover").fancybox({
-                'type' : 'image',
-                prevEffect      : 'none',
-                nextEffect      : 'none'
-                <?php if ($isEink) echo ", openEffect : 'none', closeEffect : 'none', helpers : {overlay : null}"; ?>
-            });
+            $(".fancycover").fancybox(fancyBoxObject (null, 'image'));
             
-            $(".fancyabout").fancybox({
-                'type' : 'ajax',
-                title           : 'COPS <?php echo VERSION ?>',
-                prevEffect      : 'none',
-                nextEffect      : 'none'
-                <?php if ($isEink) echo ", openEffect : 'none', closeEffect : 'none', helpers : {overlay : null}"; ?>
-            });            
+            $(".fancyabout").fancybox(fancyBoxObject ('COPS <?php echo VERSION ?>', 'ajax'));
 <?php } ?>
             
             $(".headright").click(function(){

util.js

@@ -1,5 +1,19 @@
 var template, templateBookDetail, result;
 
+var isEink = /Kobo|Kindle|EBRD1101/i.test(navigator.userAgent);
+
+function fancyBoxObject (title, type) {
+    var out = { prevEffect      : 'none', nextEffect      : 'none' };
+    if (isEink) {
+        out ["openEffect"] = 'none';
+        out ["closeEffect"] = 'none';
+        out ["helper"] = { overlay : null };
+    }
+    if (title) out ["title"] = title;
+    if (type) out ["type"] = type;
+    return out;
+}
+
 function htmlEscape(str) {
     return String(str)
             .replace(/&/g, '&')