[bug] fix link focus on edit or remove

Author: zombieFox

css/link.css

@@ -54,6 +54,7 @@
 .link-item:hover {
   transform: scale(1.05);
   z-index: 2;
+  outline: 0;
 }
 
 .link-item:active {

js/link.js

@@ -4,8 +4,17 @@ var link = (function() {
 
   var _returnToPreviousFocusLink = function() {
     if (_previousFocusLink != null) {
-      helper.eA(".link-panel-back")[_previousFocusLink].querySelectorAll(".link-control-item")[0].focus();
-      _previousFocusLink = null
+      var linkPanelFront = helper.eA(".link-panel-front");
+      if (linkPanelFront.length > 0) {
+        if (_previousFocusLink >= 0) {
+          linkPanelFront[_previousFocusLink].focus();
+        } else {
+          linkPanelFront[0].focus();
+        };
+      } else {
+        helper.e("body").focus();
+      };
+      _previousFocusLink = null;
     };
   };
 
@@ -129,11 +138,26 @@ var link = (function() {
   };
 
   var remove = function(bookmarkData) {
-    bookmarks.remove(bookmarkData);
-    _checkCount();
-    data.save();
-    clear();
-    render.item.all();
+    modal.render({
+      heading: "Remove " + bookmarkData.name + " bookmark",
+      content: "Are you sure you want to remove this bookmark? This can not be undone.",
+      successAction: function() {
+        _previousFocusLink = _previousFocusLink - 1;
+        bookmarks.remove(bookmarkData);
+        _checkCount();
+        data.save();
+        clear();
+        render.item.all();
+        control.dependents();
+        control.render();
+        _returnToPreviousFocusLink();
+      },
+      cancelAction: function() {
+        _returnToPreviousFocusLink();
+      },
+      actionText: "Remove",
+      size: "small"
+    });
   };
 
   var _checkCount = function() {
@@ -438,18 +462,8 @@ var link = (function() {
       edit(data);
     }, false);
     linkRemove.addEventListener("click", function() {
-      modal.render({
-        heading: "Remove " + data.name + " bookmark",
-        content: "Are you sure you want to remove this bookmark? This can not be undone.",
-        successAction: function() {
-          remove(data);
-          control.dependents();
-          control.render();
-        },
-        actionText: "Remove",
-        cancelText: "Cancel",
-        size: "small"
-      });
+      _previousFocusLink = index;
+      remove(data);
     }, false);
 
     return linkItem;

js/version.js

@@ -1,7 +1,7 @@
 var version = (function() {
 
   // version is normally bumped when the state needs changing or any new functionality is added
-  var current = "3.10.2";
+  var current = "3.10.3";
 
   var compare = function(a, b) {
     var pa = a.split(".");