Review comments

Author: ficristo

src/project/FileTreeView.js

@@ -114,35 +114,36 @@ define(function (require, exports, module) {
     /**
      * @private
      *
-     * Create an appropriate div based thickness to indent the tree correctly.
+     * Create an appropriate div based "thickness" to indent the tree correctly.
      *
-     * @param {int} depth - The depthness of the current node.
-     * @return {ReactComponent} - The resulting div.
+     * @param {int} depth The depth of the current node.
+     * @return {ReactComponent} The resulting div.
      */
     function _createThickness(depth) {
         return DOM.div({
             className: "thickness",
             style: {
-                width: (INDENTATION_WIDTH * depth) + "px"
+                display: "inline-block",
+                width: INDENTATION_WIDTH * depth
             }
         });
     }
 
     /**
      * @private
      *
-     * Create the arrow icons used for the folders indented correctly.
+     * Create, and indent correctly, the arrow icons used for the folders.
      *
-     * @param {int} depth - The depthness of the current node.
-     * @return {ReactComponent} - The resulting ins.
+     * @param {int} depth The depth of the current node.
+     * @return {ReactComponent} The resulting ins.
      */
     function _createAlignedIns(depth) {
         return DOM.ins({
             className: "jstree-icon",
             style: {
-                marginLeft: (INDENTATION_WIDTH * depth) + "px"
+                marginLeft: INDENTATION_WIDTH * depth
             }
-        }, " ");
+        });
     }
 
     /**
@@ -503,8 +504,9 @@ define(function (require, exports, module) {
                 },
                 DOM.ins({
                     className: "jstree-icon"
-                }, " "),
+                }),
             ];
+
             var thickness = _createThickness(this.props.depth);
 
             if (this.props.entry.get("rename")) {

src/styles/jsTreeTheme.less

@@ -46,7 +46,6 @@ li.jstree-closed > ul { display:none; }
 .jstree-brackets .jstree-open > ins { background-position:-72px 0; }
 .jstree-brackets .jstree-closed > ins { background-position:-54px 0; }
 .jstree-brackets .jstree-leaf > ins { background-position:-36px 0; }
-.jstree-brackets .thickness { display: inline-block; }
 
 .jstree-brackets li a:hover {
     text-decoration: none;