Merge pull request #586 from mariroz/wb-link-in-wb

wallabag a link in wallabag now in default theme, issue #505

8 files changed, 163 insertions, 103 deletions

diff --git a/themes/baggy/_head.twig b/themes/baggy/_head.twig
index 144bcaa6..acc9fa3d 100755
--- a/themes/baggy/_head.twig
+++ b/themes/baggy/_head.twig
@@ -11,4 +11,5 @@
         <script src="{{ poche_url }}/themes/{{ constant('DEFAULT_THEME') }}/js/autoClose.js"></script>
         <script src="{{ poche_url }}/themes/{{theme}}/js/jquery.cookie.js"></script>
         <script src="{{ poche_url }}/themes/{{theme}}/js/init.js"></script>
+        <script src="{{ poche_url }}/themes/{{ constant('DEFAULT_THEME') }}/js/saveLink.js"></script>
         <script src="{{ poche_url }}/themes/{{ constant('DEFAULT_THEME') }}/js/closeMessage.js"></script>
diff --git a/themes/baggy/js/init.js b/themes/baggy/js/init.js
index da644fbe..00470fbf 100755
--- a/themes/baggy/js/init.js
+++ b/themes/baggy/js/init.js
@@ -1,10 +1,7 @@
 $.fn.ready(function() {
 
   var $listmode = $('#listmode'),
-      $listentries = $("#list-entries"),
-      $bagit = $('#bagit'),
-      $bagitForm = $('#bagit-form');
-      $bagitFormForm = $('#bagit-form-form');
+      $listentries = $("#list-entries");
 
   /* ==========================================================================
      Menu
@@ -48,79 +45,4 @@ $.fn.ready(function() {
     $listmode.addClass("tablemode");
   }
 
-  /* ==========================================================================
-    bag it link and close button
-    ========================================================================== */
-
-  function toggleSaveLinkForm(url) {
-    $bagit.toggleClass("active-current");
-    $bagitForm.toggle();
-    $('#content').toggleClass("opacity03");
-    if (url !== 'undefined' && url) {
-      $('#plainurl').val(url);
-    }
-    $('#plainurl').focus();
-  }
-
-  $bagit.click(function(){
-    toggleSaveLinkForm();
-  });
-
-  $("#bagit-form-close").click(function(){
-    toggleSaveLinkForm();
-  });
-
-
-  //send "bag it link" form request via ajax
-  $bagitFormForm.submit( function(event) {
-    $bagitFormForm.css("cursor", "wait");
-    $("#add-link-result").empty();
-
-    $.ajax({
-        type: $bagitFormForm.attr('method'),
-        url: $bagitFormForm.attr('action'),
-        data: $bagitFormForm.serialize(),
-        success: function(data) {
-          $('#add-link-result').html("Done!");
-          $('#plainurl').val('');
-          $('#plainurl').blur('');
-          $bagitFormForm.css("cursor", "auto");
-          //setTimeout( function() { toggleSaveLinkForm(); }, 1000); //close form after 1000 delay
-        },
-        error: function(data) {
-          $('#add-link-result').html("Failed!");
-          $bagitFormForm.css("cursor", "auto");
-        }
-    });
-
-    event.preventDefault();
-  });
-
-  /* ==========================================================================
-    Keyboard gestion
-    ========================================================================== */
-
-  $(window).keydown(function(e){
-    if ( ( e.target.tagName.toLowerCase() !== 'input' && e.keyCode == 83 ) || e.keyCode == 27 ) {
-      toggleSaveLinkForm();
-      return false;
-    }
-  });
-
-  /* ==========================================================================
-  Process all links inside an article
-  ========================================================================== */
-
-  $("article a[href^='http']").after(function() {
-        return " <a href=\"" + $(this).attr('href') + "\" class=\"add-to-wallabag-link-after\" alt=\"add to wallabag\" title=\"add to wallabag\">w</a> ";
-  });
-
-  $(".add-to-wallabag-link-after").click(function(event){
-    toggleSaveLinkForm($(this).attr('href'));
-    event.preventDefault();
-  });
-
-
-
-
 });
diff --git a/themes/default/_head.twig b/themes/default/_head.twig
index f310e420..96a6ab33 100644
--- a/themes/default/_head.twig
+++ b/themes/default/_head.twig
@@ -9,4 +9,5 @@
         <link rel="stylesheet" href="{{ poche_url }}/themes/default/css/print.css" media="print">
         <script src="{{ poche_url }}/themes/default/js/jquery-2.0.3.min.js"></script>
         <script src="{{ poche_url }}/themes/default/js/autoClose.js"></script>
-        <script src="{{ poche_url }}/themes/default/js/closeMessage.js"></script>
\ No newline at end of file
+        <script src="{{ poche_url }}/themes/default/js/closeMessage.js"></script>
+        <script src="{{ poche_url }}/themes/default/js/saveLink.js"></script>
diff --git a/themes/default/_menu.twig b/themes/default/_menu.twig
index 0e7dd0a7..0daa0b03 100644
--- a/themes/default/_menu.twig
+++ b/themes/default/_menu.twig
@@ -3,11 +3,11 @@
                 <li><a href="./?view=fav" {% if view == 'fav' %}class="current"{% endif %}>{% trans "favorites" %}</a></li>
                 <li><a href="./?view=archive" {% if view == 'archive' %}class="current"{% endif %}>{% trans "archive" %}</a></li>
                 <li><a href="./?view=tags" {% if view == 'tags' %}class="current"{% endif %}>{% trans "tags" %}</a></li>
-                <li><a href="javascript: void(null);" id="pocheit">{% trans "save a link" %}</a><span id="pocheit-arrow"></span></li>
-                                <li><a href="javascript: void(null);" id="search">{% trans "search" %}</a><span id="search-arrow"></span></li>
+                <li><a href="javascript: void(null);" id="bagit">{% trans "save a link" %}</a><span id="bagit-arrow"></span></li>
+                <li><a href="javascript: void(null);" id="search">{% trans "search" %}</a><span id="search-arrow"></span></li>
                 <li><a href="./?view=config" {% if view == 'config' %}class="current"{% endif %}>{% trans "config" %}</a></li>
                 <li><a href="./?logout" title="{% trans "logout" %}">{% trans "logout" %}</a></li>
             </ul>
             {% include '_pocheit-form.twig' %}
-                        {% include '_search-form.twig' %}
+            {% include '_search-form.twig' %}
 
diff --git a/themes/default/_pocheit-form.twig b/themes/default/_pocheit-form.twig
index 13096159..8c982db0 100755
--- a/themes/default/_pocheit-form.twig
+++ b/themes/default/_pocheit-form.twig
@@ -1,22 +1,8 @@
-<div id="pocheit-form" class="messages info">
-    <center>
-    <form method="get" action="index.php">
-        <input required placeholder="example.com/article" class="addurl" id="plainurl" name="plainurl" type="url" />
-        <input type="submit" value="{% trans "save link!" %}" />
+<div id="bagit-form" class="messages info">
+    <a href="javascript: void(null);" id="bagit-form-close">&nbsp;</a>
+    <form method="get" action="index.php" id="bagit-form-form">
+      <input required placeholder="example.com/article" class="addurl" id="plainurl" name="plainurl" type="url" />
+      <input type="submit" value="{% trans "save link!" %}" />
+      <div id="add-link-result"></div>
     </form>
-    </center>
 </div>
-<script type="text/javascript">
-    $(document).ready(function() {
-
-        $("#pocheit-form").hide();
-
-        $("#pocheit").click(function(){
-            $("#pocheit-form").toggle();
-            $("#pocheit").toggleClass("current");
-            $("#pocheit-arrow").toggleClass("arrow-down");
-        });
-
-
-    });
-</script>
diff --git a/themes/default/css/style.css b/themes/default/css/style.css
index b3d300ed..e536ac6b 100755
--- a/themes/default/css/style.css
+++ b/themes/default/css/style.css
@@ -355,3 +355,50 @@ a.reading-time span {
   vertical-align: top;
 }
 
+
+/* ==========================================================================
+        "save a link" popup div related styles
+        ========================================================================== */
+
+#bagit-form {
+  display: none;
+  padding-left: 30px;
+  width: 450px;
+
+}
+
+a#bagit-form-close {
+  color: #FFF;
+  display: inline-block;
+  float: right;
+  background: url("../img/messages/close.png") no-repeat scroll 0 0 rgba(0, 0, 0, 0);
+  height: 16px;
+  margin: -14px -8px 0 0;
+  width: 16px;
+  text-decoration: none;
+}
+
+
+.add-to-wallabag-link-after {
+  background-color: #000;
+  color: #fff;
+  padding: 0 4px 1px 3px;
+  font-weight: bold;
+  font-size: 0.7em;
+  border-radius: 4px;
+}
+.add-to-wallabag-link-after:hover, .add-to-wallabag-link-after:active {
+  color: #fff;
+}
+.add-to-wallabag-link-after:visited {
+  color: #999;
+}
+
+#add-link-result {
+  display: inline;
+  padding-left: 10px;
+}
+
+.opacity03 {
+  /*opacity: 0.3;*/
+}
diff --git a/themes/default/js/saveLink.js b/themes/default/js/saveLink.js
new file mode 100755
index 00000000..ccc00d1e
--- /dev/null
+++ b/themes/default/js/saveLink.js
@@ -0,0 +1,101 @@
+$.fn.ready(function() {
+
+  var $bagit = $('#bagit'),
+      $bagitForm = $('#bagit-form'),
+      $bagitFormForm = $('#bagit-form-form');
+
+  /* ==========================================================================
+   bag it link and close button
+   ========================================================================== */
+
+  function toggleSaveLinkForm(url, event) {
+    $("#add-link-result").empty();
+
+    $bagit.toggleClass("active-current");
+
+    //only if bagiti link is not presented on page
+    if ( $bagit.length === 0 ) {
+      if ( event !== 'undefined' && event ) {
+        $bagitForm.css( {position:"absolute", top:event.pageY, left:event.pageX-200});
+      }
+      else {
+        $bagitForm.css( {position:"relative", top:"auto", left:"auto"});
+      }
+    }
+
+    $bagitForm.toggle();
+    $('#content').toggleClass("opacity03");
+    if (url !== 'undefined' && url) {
+      $('#plainurl').val(url);
+    }
+    $('#plainurl').focus();
+  }
+
+
+  $bagit.click(function(){
+    $bagit.toggleClass("current");
+    $("#bagit-arrow").toggleClass("arrow-down");
+    toggleSaveLinkForm();
+  });
+
+  $("#bagit-form-close").click(function(){
+    $bagit.removeClass("current");
+    $("#bagit-arrow").removeClass("arrow-down");
+    toggleSaveLinkForm();
+  });
+
+
+  //send "bag it link" form request via ajax
+  $bagitFormForm.submit( function(event) {
+    $("body").css("cursor", "wait");
+    $("#add-link-result").empty();
+
+    $.ajax({
+      type: $bagitFormForm.attr('method'),
+      url: $bagitFormForm.attr('action'),
+      data: $bagitFormForm.serialize(),
+      success: function(data) {
+        $('#add-link-result').html("Done!");
+        $('#plainurl').val('');
+        $('#plainurl').blur('');
+        $("body").css("cursor", "auto");
+        //setTimeout( function() { toggleSaveLinkForm(); }, 1000); //close form after 1000 delay
+      },
+      error: function(data) {
+        $('#add-link-result').html("Failed!");
+        $("body").css("cursor", "auto");
+      }
+    });
+
+    event.preventDefault();
+  });
+
+  /* ==========================================================================
+   Keyboard gestion
+   ========================================================================== */
+
+  $(window).keydown(function(e){
+    if ( ( e.target.tagName.toLowerCase() !== 'input' && e.keyCode == 83 ) || e.keyCode == 27 ) {
+      $bagit.removeClass("current");
+      $("#bagit-arrow").removeClass("arrow-down");
+      toggleSaveLinkForm();
+      return false;
+    }
+  });
+
+  /* ==========================================================================
+   Process all links inside an article
+   ========================================================================== */
+
+  $("article a[href^='http']").after(function() {
+    return " <a href=\"" + $(this).attr('href') + "\" class=\"add-to-wallabag-link-after\" alt=\"add to wallabag\" title=\"add to wallabag\">w</a> ";
+  });
+
+  $(".add-to-wallabag-link-after").click(function(event){
+    toggleSaveLinkForm($(this).attr('href'), event);
+    event.preventDefault();
+  });
+
+});
+
+
diff --git a/themes/default/view.twig b/themes/default/view.twig
index 916abe0d..2be0c33c 100644
--- a/themes/default/view.twig
+++ b/themes/default/view.twig
@@ -1,6 +1,7 @@
 {% extends "layout.twig" %}
 {% block title %}{{ entry.title|raw }} ({{ entry.url | e | getDomain }}){% endblock %}
 {% block content %}
+        {% include '_pocheit-form.twig' %}
         <div id="article_toolbar">
             <ul>
                 <li><a href="./" title="{% trans "Return home" %}" class="tool back"><span>{% trans "Return home" %}</span></a></li>
@@ -55,3 +56,4 @@
         });
         </script>
 {% endblock %}
+