Fixed #125

Real-Gecko · Jan 31, 2018 · c3db0b6 · c3db0b6
1 parent 9093610
commit c3db0b6
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,10 +2,14 @@
 ## [2.2.0] - 2018-
 ### Fixed
 - Fixed #126
+- Fixed #125
 
 ### Added
 - Added symlink for text/csv file mimetype image to point to text/plain
 
+### Changed
+- Some code cleanup
+
 ### Updated
 - Updated Bootstrap to 3.3.7
 - Updated jQuery to 2.2.4

diff --git a/filemin-lib.pl b/filemin-lib.pl
@@ -153,7 +153,7 @@ sub get_bookmarks {
     }
     $result = '';
     foreach $bookmark(@$bookmarks) {
-        $result.= "<li><a data-item='goto'>$bookmark</a><li>";
+        $result.= "<li><a data-item='goto' data-original-title='$bookmark'>$bookmark</a><li>";
     }
     return $result;
 }

diff --git a/unauthenticated/css/filemin.css b/unauthenticated/css/filemin.css
@@ -206,6 +206,11 @@ nav.navbar {
     padding-right: 5px;
 }
 
+#bookmarks [data-item="goto"] {
+    max-width: 340px;
+    overflow: hidden;
+    text-overflow: ellipsis;
+}
 /*
 .column-field-name {
     cursor: pointer;

diff --git a/unauthenticated/js/filemin.js b/unauthenticated/js/filemin.js
@@ -642,6 +642,8 @@ $(document).ready( function () {
     PNotify.prototype.options.addclass = "stack-bottomright filemin";
 
     filemin = new Filemin();
+
+    $('[ data-item="goto" ]').tooltip({trigger: 'hover', placement: 'bottom', html: true});
 
     /* Initialize tree */
     $('#filemin-tree .tree').fancytree({

diff --git a/unauthenticated/js/spec-ops.js b/unauthenticated/js/spec-ops.js
@@ -292,7 +292,9 @@ function bookmark(tab) {
     $.post("bookmark.cgi", { 'path': tab.path })
     .done(function(response) {
         if(response.success) {
-            $('#bookmarks').append('<li><a data-item="goto">' + escapeHTML(tab.path) + '</a></li>');
+            $('#bookmarks').append('<li><a data-item="goto" data-original-title="' + escapeHTML(tab.path) + '">' + escapeHTML(tab.path) + '</a></li>');
+            $('[ data-item="goto" ]').tooltip({trigger: 'hover', placement: 'bottom', html: true});
+
             showSuccess(null, text.bookmark_added);
         } else {
             showError(null, response.error)
@@ -781,9 +783,10 @@ function manageBookmarks() {
                             });
                             $.each(lines, function(ix, el) {
                                 if(el !== '') {
-                                    $(ul).append('<li><a data-item="goto">' + el + '</a><li>');
+                                    $(ul).append('<li><a data-item="goto" data-original-title="' + el + '">' + el + '</a><li>');
                                 }
                             });
+                            $('[ data-item="goto" ]').tooltip({trigger: 'hover', placement: 'bottom', html: true});
                         } else {
                             showError(null, response.error);
                         }