Элемент "назад" в последней ячейке, URL теперь не валидится, README

Author: PowercoderJr

README.md

@@ -1,3 +1,41 @@
 # Multilevel Speed Dial
+Организуй часто посещаемые сайты в удобное меню визуальных закладок вместо
+пустой страницы браузера. Закладки можно собирать в папки, а папки - в другие
+папки. Быстро открыть закладку или папку можно с любой другой вкладки,
+используя сочетания клавиш на клавиатуре. Каждую папку можно стилизовать.
+Если не хочешь привлекать внимание к какой-либо закладке, её не обязательно
+прятать куда-нибудь подальше, а можно лишь скрыть её превью и/или заголовок
+и расположить так, как будет удобно. Структуру закладок можно легко
+синхронизировать между разными устройствами или сохранить на диск.
 
-Extension for Mozilla Firefox that adds visual bookmarks. Bookmarks can be divided into folders for orderly accessing. User can navigate through saved bookmarks by opening new tab or by using shortcut with bookmark's code.
+# Зачем этому расширению различные полномочия?
+- Хранение неограниченного количества клиентских данных:
+    - хранение информации о закладках и папках;
+    - хранение настроек;
+- Доступ к истории просмотров:
+    - составление списка предложений во время создания закладки;
+- Доступ ко вкладкам браузера:
+    - открытие закладок в существующих или новых вкладках;
+    - обработка сочетаний клавиш для навигации по закладкам с клавиатуры;
+- Доступ к данным других сайтов (не обязательно):
+    - получение информации о страницах во время создания закладок (заголовок
+      страницы, иконка, скриншот-превью);
+    - размещение на страницах элементов интерфейса для навигации с клавиатуры;
+- Сокрытие и отображение вкладок (не обязательно):
+    - запрос информации о страницах (заголовка, иконки, скриншота-превью) при
+      создании и обновлении закладок без отображения новой вкладки на панели
+      вкладок.
+
+# Навигация с помощью сочетаний клавиш
+Открыть закладку или папку (далее - элемент) можно с любой страницы используя
+сочетания клавиш клавиатуры сразу после того как страница полностью загрузится.
+Для этого следует зажать <kbd>Ctrl</kbd> и, не отпуская, набрать адрес элемента. Адрес - это
+номера папок, разделённые слешами (символ "/"), ведущие к данному элементу.
+Всё как с файлами и папками на компьютере, но в виде чисел.
+
+Итак, зажав <kbd>Ctrl</kbd>, используй следующие клавиши:
+- <kbd>0</kbd>-<kbd>9</kbd> → набор адреса;
+- <kbd>+</kbd>/<kbd>-</kbd> → перебор элементов в текущей папке;
+- <kbd>/</kbd>/<kbd>SPACE</kbd> → разделитель;
+- <kbd>Backspace</kbd> → стереть;
+- <kbd>Shift</kbd> → открыть в новой вкладке (достаточно нажать один раз).

_locales/ru/messages.json

@@ -6,7 +6,7 @@
 
     "extensionDescription":
     {
-        "message": "Организуй часто посещаемые сайты в удобное меню визуальных закладкок вместо\nпустой страницы браузера. Закладки можно собирать в папки, а папки - в другие\nпапки. Быстро открыть закладку или папку можно с любой другой вкладки,\nиспользуя сочетания клавиш на клавиатуре. Каждую папку можно стилизовать.\nЕсли не хочешь привлекать внимание к какой-либо закладке, её не обязательно\nпрятать куда-нибудь подальше, а можно лишь скрыть её превью и/или заголовок\nи расположить так, как бдует удобно. Структуру закладок можно лекго\nсинхронизировать между разными устройствами или сохранить на диск.\n\nЗачем этому расширению различные полномочия?\n- Хранение неограниченного количества клиентских данных:\n    - хранение информации о закладках и папках;\n    - хранение настроек;\n- Доступ к истории просмотров:\n    - составление списка предложений во время создания закладки;\n- Доступ ко вкладкам браузера:\n    - открытие закладок в существующих или новых вкладках;\n    - обработка сочетаний клавиш для навигации по закладкам с клавиатуры;\n- Доступ к данным других сайтов (не обязательно):\n    - получение информации о страницах во время создания закладок (заголовок\n      страницы, иконка, скриншот-превью);\n    - размещение на страницах элементов интерфейса для навигации с клавиатуры;\n- Сокрытие и отображение вкладок (не обязательно):\n    - запрос информации о страницах (заголовка, иконки, скриншота-превью) при\n      создании и обновлении закладок без отображения новой вкладки на панели\n      вкладок."
+        "message": "Организуй часто посещаемые сайты в удобное меню визуальных закладок вместо\nпустой страницы браузера. Закладки можно собирать в папки, а папки - в другие\nпапки. Быстро открыть закладку или папку можно с любой другой вкладки,\nиспользуя сочетания клавиш на клавиатуре. Каждую папку можно стилизовать.\nЕсли не хочешь привлекать внимание к какой-либо закладке, её не обязательно\nпрятать куда-нибудь подальше, а можно лишь скрыть её превью и/или заголовок\nи расположить так, как будет удобно. Структуру закладок можно легко\nсинхронизировать между разными устройствами или сохранить на диск.\n\nЗачем этому расширению различные полномочия?\n- Хранение неограниченного количества клиентских данных:\n    - хранение информации о закладках и папках;\n    - хранение настроек;\n- Доступ к истории просмотров:\n    - составление списка предложений во время создания закладки;\n- Доступ ко вкладкам браузера:\n    - открытие закладок в существующих или новых вкладках;\n    - обработка сочетаний клавиш для навигации по закладкам с клавиатуры;\n- Доступ к данным других сайтов (не обязательно):\n    - получение информации о страницах во время создания закладок (заголовок\n      страницы, иконка, скриншот-превью);\n    - размещение на страницах элементов интерфейса для навигации с клавиатуры;\n- Сокрытие и отображение вкладок (не обязательно):\n    - запрос информации о страницах (заголовка, иконки, скриншота-превью) при\n      создании и обновлении закладок без отображения новой вкладки на панели\n      вкладок."
     },
 
     "developerName":

elements/BackstepElement.js

@@ -9,7 +9,7 @@ import Element from './Element.js';
  *
  * @param   int number  Номер элемента
  */
-var BackstepElement = function(number = 1) {
+var BackstepElement = function(number) {
     Element.call(this, number);
     this.type = ElementType.BACKSTEP;
 }
@@ -28,7 +28,7 @@ export default BackstepElement;
  *                          элемента с заданными свойствами
  */
 BackstepElement.prototype.parseObj = function(data) {
-    return new BackstepElement();
+    return new BackstepElement(data.number);
 }
 
 /**

elements/Element.js

@@ -164,7 +164,6 @@ Element.prototype.onDragEnter = function(event) {
     let srcPath = JSON.parse(event.dataTransfer.getData("srcPath"));
 
     let isDragTarget = arraysEqual(currElementPath, srcPath);
-    let isDropTarget;
     if (!isOverDropTarget && !isDragTarget) {
         let date = new Date();
         dragEnterTime = date.getTime();
@@ -208,22 +207,24 @@ Element.prototype.onDragOver = function(event) {
 Element.prototype.onDrop = function(event) {
     event.preventDefault();
 
-    let currElementPathStr = Array.from(currPath);
-    currElementPathStr.push(this.number);
-    currElementPathStr = currElementPathStr.join("/");
-    let srcPath = JSON.parse(event.dataTransfer.getData("srcPath"));
-    let srcPathStr = srcPath.join("/");
-
-    let isLoopDetected = srcPathStr.startsWith(currElementPathStr);
-
-    if (this.type != ElementType.BACKSTEP && !isLoopDetected) {
-        let dstPath = Array.from(currPath);
-        dstPath.push(this.number);
-
-        swapElements(srcPath, dstPath);
-    } else {
-        event.currentTarget.style.borderStyle = "solid";
-        alert(browser.i18n.getMessage("ohnonono"));
+    if (this.type != ElementType.BACKSTEP) {
+        let currElementPathStr = Array.from(currPath);
+        currElementPathStr.push(this.number);
+        currElementPathStr = currElementPathStr.join("/");
+        let srcPath = JSON.parse(event.dataTransfer.getData("srcPath"));
+        let srcPathStr = srcPath.join("/");
+        let isLoopDetected = srcPathStr.startsWith(currElementPathStr) &&
+                srcPathStr.length > currElementPathStr.length;
+
+        if (!isLoopDetected) {
+            let dstPath = Array.from(currPath);
+            dstPath.push(this.number);
+
+            swapElements(srcPath, dstPath);
+        } else {
+            event.currentTarget.style.borderStyle = "solid";
+            alert(browser.i18n.getMessage("ohnonono"));
+        }
     }
 }
 

elements/Folder.js

@@ -58,13 +58,13 @@ var Folder = function(number, caption, rows = 3, cols = 3,
 
     let amount = this.rows * this.cols;
     this.elements = new Array(amount);
+    for (let i = 0; i < amount - 1; ++i) {
+        this.elements[i] = new Element(i + 1);
+    }
     if (this.number > 0) {
-        this.elements[0] = new BackstepElement();
+        this.elements[amount - 1] = new BackstepElement(amount);
     } else {
-        this.elements[0] = new Element(1);
-    }
-    for (let i = 1; i < amount; ++i) {
-        this.elements[i] = new Element(i + 1);
+        this.elements[amount - 1] = new Element(amount);
     }
 }
 Folder.prototype = Object.create(FilledElement.prototype);

mlsd.html

@@ -16,6 +16,7 @@
         <div id="assignmentFormContainer">
             <form id="assignmentForm" method="post">
                 <input type="hidden" id="modeTf" name="mode">
+                <input type="hidden" id="bsNumberTf" name="mode">
                 <fieldset>
                     <legend id="cellAssignmentTitle"></legend>
                     <div>
@@ -37,7 +38,8 @@
                         <legend id="bookmarkSettingsLabel"></legend>
                         <p>
                             <label id="urlLabel"></label>
-                            <input type="url" id="urlTf" name="url" size="80" placeholder="https://www.google.com/" list="urlSuggectionsDL" autocomplete="off" required>
+                            <!--<input type="url" id="urlTf" name="url" size="80" placeholder="https://www.google.com/" list="urlSuggectionsDL" autocomplete="off" required>-->
+                            <input type="text" id="urlTf" name="url" size="80" placeholder="https://www.google.com/" list="urlSuggectionsDL" autocomplete="off" required>
                             <datalist id="urlSuggectionsDL">
                             </datalist>
                         </p>

mlsd.js

@@ -80,12 +80,12 @@ window.onload = function() {
         onStorageCheckedOut(results);
         buildPage(rootFolder);
     }, onPromiseFailed);
-    browser.storage.local.get().then(function(all) { //DEBUG
+    /*browser.storage.local.get().then(function(all) { //DEBUG
         console.log("Stored data: ");
         for (let key in all) {
             console.log(key, " = ", all[key]);
         }
-    }, onPromiseFailed);
+    }, onPromiseFailed);*/
 
     /*Вставка строк из файлов локализации*/
     document.getElementById("cellAssignmentTitle").textContent = browser.i18n.getMessage("cellAssignmentTitle") + " - " + browser.i18n.getMessage("extensionName");
@@ -190,6 +190,7 @@ export function initFolderForm(folder) {
         if (newAmount < oldAmount) {
             let elements;
             let numberTf = document.getElementById("numberTf");
+            //let bsNumberTf = document.getElementById("bsNumberTf");
             if (folder) {
                 elements = folder.elements;
             } else {
@@ -198,7 +199,8 @@ export function initFolderForm(folder) {
             }
             let nBookmarks = 0;
             let nFolders = 0;
-            for (let i = newAmount; i < oldAmount; ++i) {
+            const lastSavedN = newAmount - (numberTf ? 1 : 0);
+            for (let i = lastSavedN; i < oldAmount; ++i) {
                 switch (elements[i].type) {
                     case ElementType.BOOKMARK:
                         ++nBookmarks;
@@ -228,7 +230,7 @@ export function initFolderForm(folder) {
                 const endingBookmarks = getEnding(nBookmarks);
                 const endingFolders = getEnding(nFolders);
                 document.getElementById("elemsWillBeLostLabel").textContent =
-                        browser.i18n.getMessage("elemsWillBeLost", [newAmount,
+                        browser.i18n.getMessage("elemsWillBeLost", [lastSavedN,
                         nBookmarks, endingBookmarks, nFolders, endingFolders]);
                 document.getElementById("elemsWillBeLostLabel").style.display = "initial";
             } else {
@@ -627,10 +629,18 @@ async function parseAssignmentForm(copyElems) {
         if (copyElems) {
             let src = getFolderByPath(currPath, rootFolder).elements[number - 1];
             if (src.type == ElementType.FOLDER) {
-                const bound = Math.min(src.elements.length, rows * cols);
+                let amount = rows * cols;
+                const bound = Math.min(src.elements.length, amount);
                 for (let i = 0; i < bound; ++i) {
                     result.elements[i] = src.elements[i];
                 }
+                let bsIndex = getBackstepIndex(result);
+                console.log("bsIndex = ", bsIndex, " amount = ", amount);
+                if (bsIndex >= 0 && bsIndex + 1 < amount) {
+                    result.elements[bsIndex] = new Element(bsIndex + 1);
+                    console.log("Done");
+                }
+                result.elements[amount - 1] = new BackstepElement(amount);
             }
         }
     } else {
@@ -868,7 +878,21 @@ browser.runtime.onMessage.addListener(function(msg, sender, sendResponse) {
             break;
         default:
             //DEBUG
-            console.log("Поступила какая-то команда, но мы её проигнорируем");
+            //console.log("Поступила какая-то команда, но мы её проигнорируем");
             break;
     }
 });
+
+/**
+ * Поиск элемента возврата в папке
+ *
+ * @param   Folder  folder  Папка, в которой осуществляется поиск
+ */
+function getBackstepIndex(folder) {
+    for (let i = 0; i < folder.elements.length; ++i) {
+        if (folder.elements[i].type == ElementType.BACKSTEP) {
+            return i;
+        }
+    }
+    return -1;
+}