Ver Fonte

HUE-5126 [fb] Filechooser can have some autocomplete error

Enrico Berti há 9 anos atrás
pai
commit
e0e7567

+ 3 - 4
desktop/core/src/desktop/static/desktop/css/hue3.css

@@ -931,22 +931,21 @@ div.box {
 }
 
 .filechooser-tree .editable-breadcrumb-input {
-  margin-left: 10px;
+  margin-left: 30px;
   margin-bottom: 0;
   border-radius: 2px;
   border: 1px solid #EEE;
   -webkit-box-shadow: none;
   -moz-box-shadow: none;
   box-shadow: none;
-  width: 370px;
+  width: 340px;
   height: 20px;
   line-height: 20px;
   min-height: 20px;
   font-size: 1.25em;
-  padding-left: 10px!important;
+  padding-left: 4px!important;
 }
 
-
 /*
  * jHueSelector
  */

+ 4 - 5
desktop/core/src/desktop/static/desktop/js/jquery.filechooser.js

@@ -201,7 +201,6 @@
           'position': 'absolute',
           'right': '20px',
           'background-color': '#FFF',
-          'top': '2px'
         });
 
         function slideOutInput() {
@@ -263,7 +262,7 @@
           'paddingLeft': '10px',
           'paddingRight': '10px',
           'float': 'left',
-          'width': '370px',
+          'width': '350px',
           'overflow-x': 'scroll',
           'overflow-y': 'hidden',
           'white-space': 'nowrap'
@@ -293,15 +292,15 @@
         var $hdfsAutocomplete = $('<input type="text">').addClass('editable-breadcrumb-input').val(path).hide();
 
         $scrollingBreadcrumbs.click(function (e) {
-          if ($(e.target).is('ul') || $(e.target).is('i')) {
+          if ($(e.target).is('ul') || $(e.target).hasClass('spacer')) {
             $(this).hide();
             $hdfsAutocomplete.show().focus();
           }
         });
 
         var $editBreadcrumbs = $("<li>").css('marginRight', '10px');
-        var $crumbLink = $("<a>").addClass('inactive-action');
-        $crumbLink.attr("href", "javascript:void(0)").html('<i class="fa fa-pencil"></i>').appendTo($editBreadcrumbs);
+        var $crumbLink = $("<span>").addClass('spacer');
+        $crumbLink.html('&nbsp;').appendTo($editBreadcrumbs);
         $editBreadcrumbs.appendTo($scrollingBreadcrumbs);
         if (typeof data.breadcrumbs != "undefined" && data.breadcrumbs != null) {
           var _bLength = data.breadcrumbs.length;

+ 1 - 1
desktop/core/src/desktop/static/desktop/js/jquery.hdfsautocomplete.js

@@ -249,7 +249,7 @@
               if (item.type == "dir") {
                 ico = "fa-folder";
               }
-              _currentFiles.push('<li class="hdfsAutocompleteItem" data-value="' + item.name + '"><i class="fa ' + ico + '"></i> ' + item.name + '</li>');
+              _currentFiles.push('<li class="hdfsAutocompleteItem" data-value="' + escapeOutput(item.name) + '"><i class="fa ' + ico + '"></i> ' + escapeOutput(item.name) + '</li>');
             }
           });
           window.setTimeout(function () {