From 62a994d8aaf21958680300f6e33056fbc32b77eb Mon Sep 17 00:00:00 2001
From: serapath <dev@serapath.de>
Date: Thu, 25 Jan 2018 05:30:10 +0800
Subject: [PATCH 01/30] fix file explorer update events 1/2

---
 src/app.js                     |  2 +-
 src/app/files/browser-files.js | 12 ++++++++----
 src/app/files/file-explorer.js | 33 ++++++++++++++++++---------------
 src/app/files/shared-folder.js |  1 +
 4 files changed, 28 insertions(+), 20 deletions(-)

diff --git a/src/app.js b/src/app.js
index 9db3e79058..5a15ec9817 100644
--- a/src/app.js
+++ b/src/app.js
@@ -497,7 +497,7 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
     switchFile: function (path) {
       fileManager.switchFile(path)
     },
-    event: this.event,
+    event: fileManager.event,
     currentFile: function () {
       return config.get('currentFile')
     },
diff --git a/src/app/files/browser-files.js b/src/app/files/browser-files.js
index ac0504e4aa..2e22536708 100644
--- a/src/app/files/browser-files.js
+++ b/src/app/files/browser-files.js
@@ -118,19 +118,21 @@ function Files (storage) {
   this.resolveDirectory = function (path, callback) {
     var self = this
     if (path[0] === '/') path = path.substring(1)
+    if (path[0] === '.' && path[1] === '/') path = path.substring(2)
     if (!path) return callback(null, { [self.type]: { } })
+    path = self.removePrefix('' + (path || ''))
     var filesList = {}
     var tree = {}
     // add r/w filesList to the list
     storage.keys().forEach((path) => {
       // NOTE: as a temporary measure do not show the config file
       if (path !== '.remix.config') {
-        filesList[self.type + '/' + path] = false
+        filesList[path] = false
       }
     })
     // add r/o files to the list
     Object.keys(readonly).forEach((path) => {
-      filesList[self.type + '/' + path] = true
+      filesList[path] = true
     })
 
     Object.keys(filesList).forEach(function (path) {
@@ -139,7 +141,7 @@ function Files (storage) {
         '/content': self.get(path)
       })
     })
-    return callback(null, tree[path] || {})
+    return callback(null, tree)
     function hashmapize (obj, path, val) {
       var nodes = path.split('/')
       var i = 0
@@ -155,7 +157,9 @@ function Files (storage) {
   }
 
   this.removePrefix = function (path) {
-    return path.indexOf(this.type + '/') === 0 ? path.replace(this.type + '/', '') : path
+    path = path.indexOf(this.type) === 0 ? path.replace(this.type, '') : path
+    if (path[0] === '/') return path.substring(1)
+    return path
   }
 
   // rename .browser-solidity.json to .remix.config
diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 15baaf437d..29377b092c 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -53,6 +53,8 @@ var css = csjs`
 `
 module.exports = fileExplorer
 
+var focusElement = null
+
 function fileExplorer (appAPI, files) {
   var self = this
   this.files = files
@@ -101,7 +103,7 @@ function fileExplorer (appAPI, files) {
       }
     },
     formatSelf: function formatSelf (key, data, li) {
-      var isRoot = data.path.indexOf('/') === -1
+      var isRoot = data.path === self.files.type
       return yo`<label class="${data.children ? css.folder : css.file}"
         data-path="${data.path}"
         style="${isRoot ? 'font-weight:bold;' : ''}"
@@ -125,7 +127,7 @@ function fileExplorer (appAPI, files) {
       if (!fileTree) return
       var newTree = normalize(path, fileTree)
       var tree = self.treeView.renderProperties(newTree, false)
-      childrenContainer.appendChild(tree)
+      ;[...tree.children].forEach(child => childrenContainer.appendChild(child))
     })
   })
 
@@ -143,7 +145,6 @@ function fileExplorer (appAPI, files) {
       <i class="fa fa-trash" aria-hidden="true"></i>
     </span>
   `
-
   appAPI.event.register('currentFileChanged', (newFile, explorer) => {
     if (explorer === files) {
       fileFocus(newFile)
@@ -157,7 +158,6 @@ function fileExplorer (appAPI, files) {
   fileEvents.register('fileAdded', fileAdded)
 
   var filepath = null
-  var focusElement = null
   var textUnderEdit = null
   var textInRename = false
 
@@ -335,19 +335,20 @@ function fileExplorer (appAPI, files) {
   function fileFocus (path) {
     if (filepath === path) return
     filepath = path
-    var el = getElement(filepath)
-    expandPathTo(el)
-    setTimeout(function focusNode () { el.click() }, 0)
+    setTimeout(function focusNode () {
+      // @TODO: fix file tree expand logic to account for async remixd loading logic 
+      var el = getElement(filepath)
+      expandPathTo(el)
+      setTimeout(function focusNode () { el.click() }, 0)
+    }, 0)
   }
 
   function fileRemoved (filepath) {
-    // @TODO: only important if currently visible in TreeView
     var li = getElement(filepath)
     if (li) li.parentElement.removeChild(li)
   }
 
   function fileRenamed (oldName, newName, isFolder) {
-    // @TODO: only important if currently visible in TreeView
     var li = getElement(oldName)
     if (li) {
       oldName = oldName.split('/')
@@ -369,13 +370,15 @@ function fileExplorer (appAPI, files) {
   }
 
   function fileAdded (filepath) {
-    // @TODO: only important if currently visible in TreeView
-    self.files.resolveDirectory('./', (error, files) => {
+    var folderpath = filepath.split('/').slice(0, -1).join('/')
+    self.files.resolveDirectory(folderpath, (error, fileTree) => {
       if (error) console.error(error)
-      var element = self.treeView.render(files)
-      element.className = css.fileexplorer
-      self.element.parentElement.replaceChild(element, self.element)
-      self.element = element
+      if (!fileTree) return
+      fileTree = normalize(folderpath, fileTree)
+      var newTree = self.treeView.renderProperties(fileTree, false)
+      var currentTree = self.treeView.nodeAt(folderpath)
+      currentTree.innerHTML = ''
+      ;[...newTree.children].forEach(child => currentTree.appendChild(child))
     })
   }
 }
diff --git a/src/app/files/shared-folder.js b/src/app/files/shared-folder.js
index 6b82ffa349..a880202afb 100644
--- a/src/app/files/shared-folder.js
+++ b/src/app/files/shared-folder.js
@@ -159,6 +159,7 @@ module.exports = class SharedFolder {
   resolveDirectory (path, callback) {
     var self = this
     if (path[0] === '/') path = path.substring(1)
+    if (path[0] === '.' && path[1] === '/') path = path.substring(2)
     if (!path) return callback(null, { [self.type]: { } })
     path = self.removePrefix('' + (path || ''))
     self.remixd.dir(path, callback)

From d4252961e3f3b23ebd6baad94bb41a6f01f312a5 Mon Sep 17 00:00:00 2001
From: serapath <dev@serapath.de>
Date: Tue, 30 Jan 2018 07:29:08 +0800
Subject: [PATCH 02/30] fix expand logic on file tab click

---
 src/app.js                     | 20 +++++---------
 src/app/files/file-explorer.js | 50 ++++++++++++++++++++++++----------
 src/app/files/fileManager.js   | 23 +++++++++-------
 3 files changed, 55 insertions(+), 38 deletions(-)

diff --git a/src/app.js b/src/app.js
index 5a15ec9817..308ebe6482 100644
--- a/src/app.js
+++ b/src/app.js
@@ -498,9 +498,7 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
       fileManager.switchFile(path)
     },
     event: fileManager.event,
-    currentFile: function () {
-      return config.get('currentFile')
-    },
+    config: config,
     currentContent: function () {
       return editor.get(config.get('currentFile'))
     },
@@ -516,18 +514,14 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
 
   filePanel.event.register('resize', delta => self._adjustLayout('left', delta))
 
-  var previouslyOpenedFile = config.get('currentFile')
-  if (previouslyOpenedFile) {
+  setTimeout(function initOpenFile () {
+    var previouslyOpenedFile = config.get('currentFile') || 'browser/ballot.sol'
+    config.set('currentFile', null)
     filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
-      if (!error && content) {
-        fileManager.switchFile(previouslyOpenedFile)
-      } else {
-        fileManager.switchFile()
-      }
+      if (!error && content) fileManager.switchFile(previouslyOpenedFile)
+      else fileManager.switchFile('browser/ballot.sol')
     })
-  } else {
-    fileManager.switchFile()
-  }
+  }, 0)
 
   // ----------------- Renderer -----------------
   var rendererAPI = {
diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 29377b092c..9cd8b1ae94 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -149,6 +149,8 @@ function fileExplorer (appAPI, files) {
     if (explorer === files) {
       fileFocus(newFile)
     } else {
+      var currentFile = appAPI.config.get('currentFile')
+      if (currentFile === newFile) return
       unfocus(focusElement)
     }
   })
@@ -157,7 +159,6 @@ function fileExplorer (appAPI, files) {
   fileEvents.register('fileRenamedError', fileRenamedError)
   fileEvents.register('fileAdded', fileAdded)
 
-  var filepath = null
   var textUnderEdit = null
   var textInRename = false
 
@@ -332,15 +333,41 @@ function fileExplorer (appAPI, files) {
     })
   }
 
+  function expandPathTo (filePath, cb) {
+    var label = self.element.querySelector(`label[data-path="${filePath}"]`)
+    if (label) cb()
+    var arr = filePath.split('/')
+    var filename = arr.pop()
+    var path = arr.shift()
+    var fullpath = path
+    expand(fullpath, path, function next () {
+      path = arr.shift()
+      if (path) {
+        fullpath += '/' + path
+        expand(fullpath, path, next)
+      } else {
+        var label = self.element.querySelector(`label[data-path$="${filename}"]`)
+        focus.call(getLiFrom(label), {})
+        cb()
+      }
+    })
+    function expand (path, step, next) {
+      label = self.element.querySelector(`label[data-path="${step}"]`) ||
+        self.element.querySelector(`label[data-path="${path}"]`)
+      if (label) var li = label.parentElement.parentElement.parentElement
+      if (!label || path === step && !li.children[1].children.length) {
+        var currentTree = self.treeView.nodeAt(path)
+        var caret = currentTree.parentElement.firstElementChild.firstElementChild
+        caret.click()
+        next()
+      } else next()
+    }
+  }
+
   function fileFocus (path) {
+    var filepath = appAPI.config.get('currentFile')
     if (filepath === path) return
-    filepath = path
-    setTimeout(function focusNode () {
-      // @TODO: fix file tree expand logic to account for async remixd loading logic 
-      var el = getElement(filepath)
-      expandPathTo(el)
-      setTimeout(function focusNode () { el.click() }, 0)
-    }, 0)
+    expandPathTo(path, function () { appAPI.config.set('currentFile', path) })
   }
 
   function fileRemoved (filepath) {
@@ -435,13 +462,6 @@ function removeSubtree (files, path, isFolder) {
   if (isFolder) files.remove(path)
 }
 
-function expandPathTo (li) {
-  while ((li = li.parentElement.parentElement) && li.tagName === 'LI') {
-    var caret = li.firstChild.firstChild
-    if (caret.classList.contains('fa-caret-right')) caret.click() // expand
-  }
-}
-
 fileExplorer.prototype.init = function () {
   var self = this
   self.files.resolveDirectory('/', (error, files) => {
diff --git a/src/app/files/fileManager.js b/src/app/files/fileManager.js
index 9a5875b650..8acaaa2a01 100644
--- a/src/app/files/fileManager.js
+++ b/src/app/files/fileManager.js
@@ -47,6 +47,7 @@ class FileManager {
         self.switchFile(Object.keys(self.tabbedFiles)[0])
       } else {
         opt.editor.displayEmptyReadOnlySession()
+        self.opt.config.set('currentFile', '')
       }
       return false
     })
@@ -98,7 +99,6 @@ class FileManager {
 
   // Display files that have already been selected
   refreshTabs (newfile) {
-    var self = this
     if (newfile) {
       this.tabbedFiles[newfile] = newfile
     }
@@ -109,14 +109,14 @@ class FileManager {
     for (var file in this.tabbedFiles) {
       $filesEl.append(yo`<li class="file"><span class="name">${file}</span><span class="remove"><i class="fa fa-close"></i></span></li>`)
     }
-    var currentFileOpen = !!this.opt.config.get('currentFile')
 
-    if (currentFileOpen) {
-      var active = $('#files .file').filter(function () { return $(this).find('.name').text() === self.opt.config.get('currentFile') })
-      active.addClass('active')
-    }
-    $('#input').toggle(currentFileOpen)
-    $('#output').toggle(currentFileOpen)
+    var active = $('#files .file').filter(function () {
+      return $(this).find('.name').text() === newfile
+    })
+    if (active.length) active.addClass('active')
+    else this.switchFile()
+    // $('#input').toggle(active)
+    $('#output').toggle(active)
   }
 
   switchFile (file) {
@@ -128,12 +128,15 @@ class FileManager {
         if (fileList.length) {
           file = fileList[0]
           if (file) _switchFile(file)
+        } else {
+          self.event.trigger('currentFileChanged', [])
         }
       })
     } else _switchFile(file)
-    function _switchFile () {
+    function _switchFile (file) {
       self.saveCurrentFile()
-      self.opt.config.set('currentFile', file)
+      var currentFile = self.opt.config.get('currentFile')
+      if (file === currentFile) return
       self.refreshTabs(file)
       self.fileProviderOf(file).get(file, (error, content) => {
         if (error) {

From 2940c77db077ae954105ee64df8b923e120a9130 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Tue, 30 Jan 2018 12:25:33 +0100
Subject: [PATCH 03/30] fix ballot.sol load

---
 src/app.js                   | 12 ++++--------
 src/app/files/fileManager.js |  2 --
 2 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/src/app.js b/src/app.js
index 308ebe6482..b7fb76bc6b 100644
--- a/src/app.js
+++ b/src/app.js
@@ -514,14 +514,10 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
 
   filePanel.event.register('resize', delta => self._adjustLayout('left', delta))
 
-  setTimeout(function initOpenFile () {
-    var previouslyOpenedFile = config.get('currentFile') || 'browser/ballot.sol'
-    config.set('currentFile', null)
-    filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
-      if (!error && content) fileManager.switchFile(previouslyOpenedFile)
-      else fileManager.switchFile('browser/ballot.sol')
-    })
-  }, 0)
+  var previouslyOpenedFile = config.get('currentFile')
+  filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
+    if (!error && content) fileManager.switchFile(previouslyOpenedFile)
+  })
 
   // ----------------- Renderer -----------------
   var rendererAPI = {
diff --git a/src/app/files/fileManager.js b/src/app/files/fileManager.js
index 8acaaa2a01..33c99accc5 100644
--- a/src/app/files/fileManager.js
+++ b/src/app/files/fileManager.js
@@ -135,8 +135,6 @@ class FileManager {
     } else _switchFile(file)
     function _switchFile (file) {
       self.saveCurrentFile()
-      var currentFile = self.opt.config.get('currentFile')
-      if (file === currentFile) return
       self.refreshTabs(file)
       self.fileProviderOf(file).get(file, (error, content) => {
         if (error) {

From a964f7330f61fba77efbc1d0ee76d8460d13d192 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 10:52:09 +0100
Subject: [PATCH 04/30] remove expandPathTo

---
 src/app/files/browser-files.js |   7 +--
 src/app/files/file-explorer.js | 103 +++++----------------------------
 2 files changed, 15 insertions(+), 95 deletions(-)

diff --git a/src/app/files/browser-files.js b/src/app/files/browser-files.js
index 2e22536708..8692af5fae 100644
--- a/src/app/files/browser-files.js
+++ b/src/app/files/browser-files.js
@@ -118,9 +118,8 @@ function Files (storage) {
   this.resolveDirectory = function (path, callback) {
     var self = this
     if (path[0] === '/') path = path.substring(1)
-    if (path[0] === '.' && path[1] === '/') path = path.substring(2)
     if (!path) return callback(null, { [self.type]: { } })
-    path = self.removePrefix('' + (path || ''))
+    path = self.removePrefix(path)
     var filesList = {}
     var tree = {}
     // add r/w filesList to the list
@@ -157,9 +156,7 @@ function Files (storage) {
   }
 
   this.removePrefix = function (path) {
-    path = path.indexOf(this.type) === 0 ? path.replace(this.type, '') : path
-    if (path[0] === '/') return path.substring(1)
-    return path
+    return path.indexOf(this.type + '/') === 0 ? path.replace(this.type + '/', '') : path
   }
 
   // rename .browser-solidity.json to .remix.config
diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 9cd8b1ae94..dddcae3e4c 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -57,6 +57,7 @@ var focusElement = null
 
 function fileExplorer (appAPI, files) {
   var self = this
+  this.events = new EventManager()
   this.files = files
 
   function remixdDialog () {
@@ -95,11 +96,11 @@ function fileExplorer (appAPI, files) {
       return {
         path: (tree || {}).path ? tree.path + '/' + key : key,
         children: isFile ? undefined
-        : value instanceof Array ? value.map((item, index) => ({
-          key: index, value: item
-        })) : value instanceof Object ? Object.keys(value).map(subkey => ({
-          key: subkey, value: value[subkey]
-        })) : undefined
+          : value instanceof Array ? value.map((item, index) => ({
+            key: index, value: item
+          })) : value instanceof Object ? Object.keys(value).map(subkey => ({
+            key: subkey, value: value[subkey]
+          })) : undefined
       }
     },
     formatSelf: function formatSelf (key, data, li) {
@@ -162,7 +163,6 @@ function fileExplorer (appAPI, files) {
   var textUnderEdit = null
   var textInRename = false
 
-  self.events = new EventManager()
   self.api = {}
   self.api.addFile = function addFile (file) {
     function loadFile () {
@@ -292,82 +292,16 @@ function fileExplorer (appAPI, files) {
     }
   }
 
-  function renameSubtree (label, dontcheck) {
-    var oldPath = label.dataset.path
-    var newPath = oldPath
-    newPath = newPath.split('/')
-    newPath[newPath.length - 1] = label.innerText
-    newPath = newPath.join('/')
-    if (!dontcheck) {
-      var allPaths = Object.keys(files.list())
-      for (var i = 0, len = allPaths.length, path, err; i < len; i++) {
-        path = allPaths[i]
-        if (files.isReadOnly(path)) {
-          err = 'path contains readonly elements'
-          break
-        } else if (path.indexOf(newPath) === 0) {
-          err = 'new path is conflicting with another existing path'
-          break
-        }
-      }
-    }
-    if (err) {
-      modalDialogCustom.alert(`could not rename - ${err}`)
-      label.innerText = textUnderEdit
-    } else {
-      textUnderEdit = label.innerText
-      updateAllLabels([getElement(oldPath)], oldPath, newPath)
-    }
-  }
-
-  function updateAllLabels (lis, oldPath, newPath) {
-    lis.forEach(function (li) {
-      var label = getLabelFrom(li)
-      var path = label.dataset.path
-      var newName = path.replace(oldPath, newPath)
-      label.dataset.path = newName
-      var ul = li.lastChild
-      if (ul.tagName === 'UL') {
-        updateAllLabels([...ul.children], oldPath, newPath)
-      }
-    })
-  }
 
-  function expandPathTo (filePath, cb) {
-    var label = self.element.querySelector(`label[data-path="${filePath}"]`)
-    if (label) cb()
-    var arr = filePath.split('/')
-    var filename = arr.pop()
-    var path = arr.shift()
-    var fullpath = path
-    expand(fullpath, path, function next () {
-      path = arr.shift()
-      if (path) {
-        fullpath += '/' + path
-        expand(fullpath, path, next)
-      } else {
-        var label = self.element.querySelector(`label[data-path$="${filename}"]`)
-        focus.call(getLiFrom(label), {})
-        cb()
-      }
-    })
-    function expand (path, step, next) {
-      label = self.element.querySelector(`label[data-path="${step}"]`) ||
-        self.element.querySelector(`label[data-path="${path}"]`)
-      if (label) var li = label.parentElement.parentElement.parentElement
-      if (!label || path === step && !li.children[1].children.length) {
-        var currentTree = self.treeView.nodeAt(path)
-        var caret = currentTree.parentElement.firstElementChild.firstElementChild
-        caret.click()
-        next()
-      } else next()
-    }
-  }
+  
 
   function fileFocus (path) {
     var filepath = appAPI.config.get('currentFile')
     if (filepath === path) return
-    expandPathTo(path, function () { appAPI.config.set('currentFile', path) })
+    var label = document.querySelector(`label[data-path="${path}"]`)
+    if (label) {
+      appAPI.config.set('currentFile', path)
+    }
   }
 
   function fileRemoved (filepath) {
@@ -377,19 +311,8 @@ function fileExplorer (appAPI, files) {
 
   function fileRenamed (oldName, newName, isFolder) {
     var li = getElement(oldName)
-    if (li) {
-      oldName = oldName.split('/')
-      newName = newName.split('/')
-      var index = oldName.reduce(function (idx, key, i) {
-        return oldName[i] !== newName[i] ? i : idx
-      }, undefined)
-      var newKey = newName[index]
-      var oldPath = oldName.slice(0, index + 1).join('/')
-      li = getElement(oldPath)
-      var label = getLabelFrom(li)
-      label.innerText = newKey
-      renameSubtree(label, true)
-    }
+    if (li) li.parentElement.removeChild(li)
+    fileFocus(newName)
   }
 
   function fileRenamedError (error) {

From 86db792d8f86570f79c580aabe867fac8775b3a4 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 11:23:27 +0100
Subject: [PATCH 05/30] fix fileExternakkyChanged handler

---
 src/app/files/file-explorer.js | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index dddcae3e4c..75b38ad932 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -64,8 +64,8 @@ function fileExplorer (appAPI, files) {
     return yo`<div>This file has been changed outside of Remix IDE.</div>`
   }
 
-  this.files.event.register('fileExternallyChanged', (path, content) => {
-    if (appAPI.currentFile() === path && appAPI.currentContent() !== content) {
+  this.files.event.register('fileExternallyChanged', (path, file) => {
+    if (appAPI.config.get('currentFile') === path && appAPI.currentContent() !== file.content) {
       modalDialog(path + ' changed', remixdDialog(),
         {
           label: 'Keep the content displayed in Remix',
@@ -74,7 +74,7 @@ function fileExplorer (appAPI, files) {
         {
           label: 'Replace by the new content',
           fn: () => {
-            appAPI.setText(content)
+            appAPI.setText(file.content)
           }
         }
       )
@@ -292,9 +292,6 @@ function fileExplorer (appAPI, files) {
     }
   }
 
-
-  
-
   function fileFocus (path) {
     var filepath = appAPI.config.get('currentFile')
     if (filepath === path) return

From c50c9671408abc80fca7b3a7ae185a3a2430ef42 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 20:02:19 +0100
Subject: [PATCH 06/30] remove unneeded check

---
 src/app/files/shared-folder.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/app/files/shared-folder.js b/src/app/files/shared-folder.js
index a880202afb..820b793558 100644
--- a/src/app/files/shared-folder.js
+++ b/src/app/files/shared-folder.js
@@ -159,9 +159,8 @@ module.exports = class SharedFolder {
   resolveDirectory (path, callback) {
     var self = this
     if (path[0] === '/') path = path.substring(1)
-    if (path[0] === '.' && path[1] === '/') path = path.substring(2)
     if (!path) return callback(null, { [self.type]: { } })
-    path = self.removePrefix('' + (path || ''))
+    path = self.removePrefix(path)
     self.remixd.dir(path, callback)
   }
 }

From 4df6cc687b060ef83f5498d6e850461800652090 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 20:02:35 +0100
Subject: [PATCH 07/30] remove comment

---
 src/app/files/shared-folder.js | 13 -------------
 1 file changed, 13 deletions(-)

diff --git a/src/app/files/shared-folder.js b/src/app/files/shared-folder.js
index 820b793558..e76fd0d84f 100644
--- a/src/app/files/shared-folder.js
+++ b/src/app/files/shared-folder.js
@@ -137,19 +137,6 @@ module.exports = class SharedFolder {
     return true
   }
 
-  //
-  // Tree model for files
-  // {
-  //   'a': { }, // empty directory 'a'
-  //   'b': {
-  //     'c': {}, // empty directory 'b/c'
-  //     'd': { '/readonly': true, '/content': 'Hello World' } // files 'b/c/d'
-  //     'e': { '/readonly': false, '/path': 'b/c/d' } // symlink to 'b/c/d'
-  //     'f': { '/readonly': false, '/content': '<executable>', '/mode': 0755 }
-  //   }
-  // }
-  //
-
   removePrefix (path) {
     path = path.indexOf(this.type) === 0 ? path.replace(this.type, '') : path
     if (path[0] === '/') return path.substring(1)

From c09a61ef384402eb8bc36f8219983ef0d6d0b892 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 20:03:38 +0100
Subject: [PATCH 08/30] add ensureRoot function to file-panel

---
 src/app/files/file-explorer.js | 16 ++++++++++------
 src/app/panels/file-panel.js   | 10 +++-------
 2 files changed, 13 insertions(+), 13 deletions(-)
 mode change 100755 => 100644 src/app/files/file-explorer.js

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
old mode 100755
new mode 100644
index 75b38ad932..afac4b3382
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -383,18 +383,22 @@ function removeSubtree (files, path, isFolder) {
 }
 
 fileExplorer.prototype.init = function () {
+  this.container = yo`<div></div>`
+  return this.container
+}
+
+fileExplorer.prototype.ensureRoot = function (cb) {
   var self = this
+  if (self.element && cb) return cb()
+
   self.files.resolveDirectory('/', (error, files) => {
     if (error) console.error(error)
-    var element = self.treeView.render(files)
+    var element = self.treeView.render(files, false)
     element.className = css.fileexplorer
     element.events = self.events
     element.api = self.api
-    setTimeout(function () {
-      self.element.parentElement.replaceChild(element, self.element)
+    self.container.appendChild(element)
       self.element = element
-    }, 0)
+    if (cb) cb()
   })
-  self.element = yo`<div></div>`
-  return self.element
 }
diff --git a/src/app/panels/file-panel.js b/src/app/panels/file-panel.js
index 1547f66896..55a8f4e355 100644
--- a/src/app/panels/file-panel.js
+++ b/src/app/panels/file-panel.js
@@ -73,7 +73,7 @@ function filepanel (appAPI, filesProvider) {
           </div>
           <div class=${css.treeviews}>
             <div class=${css.treeview}>${fileExplorer.init()}</div>
-            <div class="filesystemexplorer ${css.treeview}"></div>
+            <div class="filesystemexplorer ${css.treeview}">${fileSystemExplorer.init()}</div>
             <div class="swarmexplorer ${css.treeview}">${swarmExplorer.init()}</div>
             <div class="githubexplorer ${css.treeview}">${githubExplorer.init()}</div>
             <div class="gistexplorer ${css.treeview}">${gistExplorer.init()}</div>
@@ -87,7 +87,7 @@ function filepanel (appAPI, filesProvider) {
   var event = new EventManager()
   self.event = event
   var element = template()
-
+  fileExplorer.ensureRoot()
   var containerFileSystem = element.querySelector('.filesystemexplorer')
   var websocketconn = element.querySelector('.websocketconn')
   filesProvider['localhost'].remixd.event.register('connecting', (event) => {
@@ -200,7 +200,6 @@ function filepanel (appAPI, filesProvider) {
     * @param {String} txHash    - hash of the transaction
     */
   function connectToLocalhost () {
-    var container = document.querySelector('.filesystemexplorer')
     if (filesProvider['localhost'].isConnected()) {
       filesProvider['localhost'].close((error) => {
         if (error) console.log(error)
@@ -213,10 +212,7 @@ function filepanel (appAPI, filesProvider) {
               if (error) {
                 console.log(error)
               } else {
-                if (fileSystemExplorer.element && container.children.length > 0) {
-                  container.removeChild(fileSystemExplorer.element)
-                }
-                container.appendChild(fileSystemExplorer.init())
+                fileSystemExplorer.ensureRoot()
               }
             })
           }})

From ccb1534cf7989f180110e6ca7ab751769a83eb09 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 20:04:02 +0100
Subject: [PATCH 09/30] fix file added

---
 src/app/files/file-explorer.js | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)
 mode change 100644 => 100755 src/app/files/file-explorer.js

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
old mode 100644
new mode 100755
index afac4b3382..6b33738fec
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -317,15 +317,20 @@ function fileExplorer (appAPI, files) {
   }
 
   function fileAdded (filepath) {
-    var folderpath = filepath.split('/').slice(0, -1).join('/')
-    self.files.resolveDirectory(folderpath, (error, fileTree) => {
-      if (error) console.error(error)
-      if (!fileTree) return
-      fileTree = normalize(folderpath, fileTree)
-      var newTree = self.treeView.renderProperties(fileTree, false)
+    self.ensureRoot(() => {
+      var folderpath = filepath.split('/').slice(0, -1).join('/')
+
       var currentTree = self.treeView.nodeAt(folderpath)
-      currentTree.innerHTML = ''
-      ;[...newTree.children].forEach(child => currentTree.appendChild(child))
+      if (currentTree) {
+        self.files.resolveDirectory(folderpath, (error, fileTree) => {
+          if (error) console.error(error)
+          if (!fileTree) return
+          fileTree = normalize(folderpath, fileTree)
+          var newTree = self.treeView.renderProperties(fileTree, false)
+          currentTree.innerHTML = ''
+          ;[...newTree.children].forEach(child => currentTree.appendChild(child))
+        })
+      }
     })
   }
 }
@@ -398,7 +403,7 @@ fileExplorer.prototype.ensureRoot = function (cb) {
     element.events = self.events
     element.api = self.api
     self.container.appendChild(element)
-      self.element = element
+    self.element = element
     if (cb) cb()
   })
 }

From f8e5a26980a7ad22637e508ee9247ef5456c82a4 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 31 Jan 2018 20:04:19 +0100
Subject: [PATCH 10/30] fix basicReadOnlyExplorer

---
 src/app/files/basicReadOnlyExplorer.js | 53 ++++++++------------------
 1 file changed, 16 insertions(+), 37 deletions(-)

diff --git a/src/app/files/basicReadOnlyExplorer.js b/src/app/files/basicReadOnlyExplorer.js
index fa3006f53a..3ed56dcd23 100644
--- a/src/app/files/basicReadOnlyExplorer.js
+++ b/src/app/files/basicReadOnlyExplorer.js
@@ -5,7 +5,9 @@ class BasicReadOnlyExplorer {
   constructor (type) {
     this.event = new EventManager()
     this.files = {}
+    this.paths = {}
     this.normalizedNames = {} // contains the raw url associated with the displayed path
+    this.paths[type] = {}
     this.type = type
     this.readonly = true
   }
@@ -46,6 +48,18 @@ class BasicReadOnlyExplorer {
     try { // lazy try to format JSON
       content = JSON.stringify(JSON.parse(content), null, '\t')
     } catch (e) {}
+    var split = path
+    var folder = false
+    while (split.lastIndexOf('/') !== -1) {
+      var subitem = split.substring(split.lastIndexOf('/'))
+      split = split.substring(0, split.lastIndexOf('/'))
+      if (!this.paths[this.type + '/' + split]) {
+        this.paths[this.type + '/' + split] = {}
+      }
+      this.paths[this.type + '/' + split][split + subitem] = { isDirectory: folder }
+      folder = true
+    }
+    this.paths[this.type][split] = { isDirectory: folder }
     this.files[this.type + '/' + unprefixedPath] = content
     this.normalizedNames[rawPath] = path
     this.event.trigger('fileAdded', [this.type + '/' + unprefixedPath, true])
@@ -68,46 +82,11 @@ class BasicReadOnlyExplorer {
     return this.files
   }
 
-  //
-  // Tree model for files
-  // {
-  //   'a': { }, // empty directory 'a'
-  //   'b': {
-  //     'c': {}, // empty directory 'b/c'
-  //     'd': { '/readonly': true, '/content': 'Hello World' } // files 'b/c/d'
-  //     'e': { '/readonly': false, '/path': 'b/c/d' } // symlink to 'b/c/d'
-  //     'f': { '/readonly': false, '/content': '<executable>', '/mode': 0755 }
-  //   }
-  // }
-  //
-  resolveDirectory (path, callback /* (error, filesList) => { } */) {
+  resolveDirectory (path, callback) {
     var self = this
     if (path[0] === '/') path = path.substring(1)
     if (!path) return callback(null, { [self.type]: { } })
-    var tree = {}
-    // This does not include '.remix.config', because it is filtered
-    // inside list().
-    Object.keys(this.list()).forEach(function (path) {
-      hashmapize(tree, path, {
-        '/readonly': self.isReadOnly(path),
-        '/content': self.get(path)
-      })
-    })
-    return callback(null, tree[path] || {})
-    function hashmapize (obj, path, val) {
-      var nodes = path.split('/')
-      var i = 0
-
-      for (; i < nodes.length - 1; i++) {
-        var node = nodes[i]
-        if (obj[node] === undefined) {
-          obj[node] = {}
-        }
-        obj = obj[node]
-      }
-
-      obj[nodes[i]] = val
-    }
+    callback(null, this.paths[path])
   }
 
   removePrefix (path) {

From 5680b84c8c8081b754b402e48fd13d70751ca8f7 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 10:46:24 +0100
Subject: [PATCH 11/30] fix file rename

---
 src/app/files/file-explorer.js | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 6b33738fec..60d6dbc2d9 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -308,8 +308,10 @@ function fileExplorer (appAPI, files) {
 
   function fileRenamed (oldName, newName, isFolder) {
     var li = getElement(oldName)
-    if (li) li.parentElement.removeChild(li)
-    fileFocus(newName)
+    if (li) {
+      li.parentElement.removeChild(li)
+      fileAdded(newName)
+    }
   }
 
   function fileRenamedError (error) {

From f324751d82cdf740c1381feae5c85766bc79f3b4 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 11:14:07 +0100
Subject: [PATCH 12/30] simplify browser-files

---
 src/app/files/browser-files.js | 29 +----------------------------
 1 file changed, 1 insertion(+), 28 deletions(-)

diff --git a/src/app/files/browser-files.js b/src/app/files/browser-files.js
index 8692af5fae..ce645a083c 100644
--- a/src/app/files/browser-files.js
+++ b/src/app/files/browser-files.js
@@ -103,18 +103,6 @@ function Files (storage) {
     return false
   }
 
-  //
-  // Tree model for files
-  // {
-  //   'a': { }, // empty directory 'a'
-  //   'b': {
-  //     'c': {}, // empty directory 'b/c'
-  //     'd': { '/readonly': true, '/content': 'Hello World' } // files 'b/c/d'
-  //     'e': { '/readonly': false, '/path': 'b/c/d' } // symlink to 'b/c/d'
-  //     'f': { '/readonly': false, '/content': '<executable>', '/mode': 0755 }
-  //   }
-  // }
-  //
   this.resolveDirectory = function (path, callback) {
     var self = this
     if (path[0] === '/') path = path.substring(1)
@@ -135,24 +123,9 @@ function Files (storage) {
     })
 
     Object.keys(filesList).forEach(function (path) {
-      hashmapize(tree, path, {
-        '/readonly': self.isReadOnly(path),
-        '/content': self.get(path)
-      })
+      tree[path] = { isDirectory: false }
     })
     return callback(null, tree)
-    function hashmapize (obj, path, val) {
-      var nodes = path.split('/')
-      var i = 0
-      for (; i < nodes.length - 1; i++) {
-        var node = nodes[i]
-        if (obj[node] === undefined) {
-          obj[node] = {}
-        }
-        obj = obj[node]
-      }
-      obj[nodes[i]] = val
-    }
   }
 
   this.removePrefix = function (path) {

From 0d9511800304ef8efdbfef77accf6bb909f15bbe Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 11:30:09 +0100
Subject: [PATCH 13/30] set test contracts

---
 ci/browser_tests.sh             | 9 +--------
 contracts/contract1.sol         | 1 +
 contracts/contract2.sol         | 1 +
 contracts/folder1/contract1.sol | 1 +
 contracts/folder1/contract2.sol | 1 +
 5 files changed, 5 insertions(+), 8 deletions(-)
 create mode 100644 contracts/contract1.sol
 create mode 100644 contracts/contract2.sol
 create mode 100644 contracts/folder1/contract1.sol
 create mode 100644 contracts/folder1/contract2.sol

diff --git a/ci/browser_tests.sh b/ci/browser_tests.sh
index 5b4c27ce40..93da0c55cc 100755
--- a/ci/browser_tests.sh
+++ b/ci/browser_tests.sh
@@ -4,14 +4,7 @@ set -e
 
 setupRemixd () {
   mkdir remixdSharedfolder
-  cd remixdSharedfolder
-  echo "contract test1 { function get () returns (uint) { return 8; }}" > contract1.sol
-  echo "contract test2 { function get () returns (uint) { return 9; }}" > contract2.sol
-  mkdir folder1
-  cd folder1
-  echo "contract test1 { function get () returns (uint) { return 10; }}" > contract1.sol
-  echo "contract test2 { function get () returns (uint) { return 11; }}" > contract2.sol
-  cd ..
+  cd contracts
   echo 'sharing folder: '
   echo $PWD
   node ../node_modules/remixd/src/main.js -s $PWD &
diff --git a/contracts/contract1.sol b/contracts/contract1.sol
new file mode 100644
index 0000000000..4abefc018b
--- /dev/null
+++ b/contracts/contract1.sol
@@ -0,0 +1 @@
+contract test1 { function get () returns (uint) { return 8; }}
\ No newline at end of file
diff --git a/contracts/contract2.sol b/contracts/contract2.sol
new file mode 100644
index 0000000000..96b59660ba
--- /dev/null
+++ b/contracts/contract2.sol
@@ -0,0 +1 @@
+contract test2 { function get () returns (uint) { return 9; }}
\ No newline at end of file
diff --git a/contracts/folder1/contract1.sol b/contracts/folder1/contract1.sol
new file mode 100644
index 0000000000..11122d3008
--- /dev/null
+++ b/contracts/folder1/contract1.sol
@@ -0,0 +1 @@
+contract test1 { function get () returns (uint) { return 10; }}
\ No newline at end of file
diff --git a/contracts/folder1/contract2.sol b/contracts/folder1/contract2.sol
new file mode 100644
index 0000000000..04f9b2eb26
--- /dev/null
+++ b/contracts/folder1/contract2.sol
@@ -0,0 +1 @@
+contract test2 { function get () returns (uint) { return 11; }}
\ No newline at end of file

From 975eaac853ea3071e5e180c59bc99e60c51c3191 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 12:47:06 +0100
Subject: [PATCH 14/30] add setupremix script

---
 .travis.yml  | 2 +-
 package.json | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index dc03c584f3..142210d258 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -2,7 +2,7 @@ language: node_js
 node_js:
   - "7"
 script:
-  - npm run lint && npm run test && npm run downloadsolc && npm run make-mock-compiler && npm run build
+  - npm run lint && npm run test && npm run downloadsolc && npm run make-mock-compiler && npm run setupremix && npm run build
   - ./ci/browser_tests.sh
 deploy:
   - provider: script
diff --git a/package.json b/package.json
index 967adf8eec..2b2d65b988 100644
--- a/package.json
+++ b/package.json
@@ -139,6 +139,7 @@
     ]
   },
   "scripts": {
+    "setupremix": "npm run pullremix; npm run linkremixcore; npm run linkremixlib; npm run linkremixsolidity; npm run linkremixdebugger;",
     "pullremix": "mkdir remix; git clone https://github.com/ethereum/remix; cd ..",
     "linkremixcore": "cd node_modules && rm -rf remix-core && ln -s ../remix/remix-core remix-core && cd ..",
     "linkremixlib": "cd node_modules && rm -rf remix-lib && ln -s ../remix/remix-lib remix-lib && cd ..",

From 46d871020fa86f6d787ed26f4812bd910db16270 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 16:50:03 +0100
Subject: [PATCH 15/30] fix set function

---
 src/app/files/basicReadOnlyExplorer.js | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/app/files/basicReadOnlyExplorer.js b/src/app/files/basicReadOnlyExplorer.js
index 3ed56dcd23..c9c00ac00e 100644
--- a/src/app/files/basicReadOnlyExplorer.js
+++ b/src/app/files/basicReadOnlyExplorer.js
@@ -38,7 +38,8 @@ class BasicReadOnlyExplorer {
   }
 
   set (path, content, cb) {
-    this.addReadOnly(path, content)
+    var unprefixedPath = this.removePrefix(path)
+    this.addReadOnly(unprefixedPath, content)
     if (cb) cb()
     return true
   }

From dff123c875b667be150f1f8502ef11663de87396 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 17:30:20 +0100
Subject: [PATCH 16/30] use treeview API

---
 src/app/files/file-explorer.js | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 60d6dbc2d9..22789cd683 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -127,8 +127,8 @@ function fileExplorer (appAPI, files) {
       if (error) console.error(error)
       if (!fileTree) return
       var newTree = normalize(path, fileTree)
-      var tree = self.treeView.renderProperties(newTree, false)
-      ;[...tree.children].forEach(child => childrenContainer.appendChild(child))
+      newTree = self.treeView.renderProperties(newTree, false)
+      self.treeView.updateNode(path, newTree)
     })
   })
 
@@ -323,14 +323,13 @@ function fileExplorer (appAPI, files) {
       var folderpath = filepath.split('/').slice(0, -1).join('/')
 
       var currentTree = self.treeView.nodeAt(folderpath)
-      if (currentTree) {
+      if (currentTree && self.treeView.isExpanded(folderpath)) {
         self.files.resolveDirectory(folderpath, (error, fileTree) => {
           if (error) console.error(error)
           if (!fileTree) return
           fileTree = normalize(folderpath, fileTree)
           var newTree = self.treeView.renderProperties(fileTree, false)
-          currentTree.innerHTML = ''
-          ;[...newTree.children].forEach(child => currentTree.appendChild(child))
+          self.treeView.updateNode(folderpath, newTree)
         })
       }
     })

From 4ed995f9b68d34198b7c46c3b6d722e3b4e73dff Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 17:30:25 +0100
Subject: [PATCH 17/30] fix styling

---
 src/app/panels/styles/file-panel-styles.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/app/panels/styles/file-panel-styles.js b/src/app/panels/styles/file-panel-styles.js
index 6bd243422e..857985c891 100644
--- a/src/app/panels/styles/file-panel-styles.js
+++ b/src/app/panels/styles/file-panel-styles.js
@@ -77,7 +77,7 @@ var css = csjs`
   }
   .dragbar            {
     position          : absolute;
-    top               : 37px;
+    top               : 29px;
     width             : 0.5em;
     right             : 0;
     bottom            : 0;

From 26d6e3775a44399eaaf02bb2e53e18e3ec296b8d Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 19:37:17 +0100
Subject: [PATCH 18/30] test contracts

---
 contracts/folder1/contract_chrome.sol  | 1 +
 contracts/folder1/contract_firefox.sol | 1 +
 2 files changed, 2 insertions(+)
 create mode 100644 contracts/folder1/contract_chrome.sol
 create mode 100644 contracts/folder1/contract_firefox.sol

diff --git a/contracts/folder1/contract_chrome.sol b/contracts/folder1/contract_chrome.sol
new file mode 100644
index 0000000000..04f9b2eb26
--- /dev/null
+++ b/contracts/folder1/contract_chrome.sol
@@ -0,0 +1 @@
+contract test2 { function get () returns (uint) { return 11; }}
\ No newline at end of file
diff --git a/contracts/folder1/contract_firefox.sol b/contracts/folder1/contract_firefox.sol
new file mode 100644
index 0000000000..04f9b2eb26
--- /dev/null
+++ b/contracts/folder1/contract_firefox.sol
@@ -0,0 +1 @@
+contract test2 { function get () returns (uint) { return 11; }}
\ No newline at end of file

From 52914fffa34a49e375a7be068d0aaed8257009fb Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Thu, 1 Feb 2018 19:39:21 +0100
Subject: [PATCH 19/30] refactor file-explorer

---
 src/app/files/file-explorer.js | 131 ++++-----------------------------
 1 file changed, 14 insertions(+), 117 deletions(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 22789cd683..563f3c8da0 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -54,6 +54,7 @@ var css = csjs`
 module.exports = fileExplorer
 
 var focusElement = null
+var focusPath = null
 
 function fileExplorer (appAPI, files) {
   var self = this
@@ -108,8 +109,6 @@ function fileExplorer (appAPI, files) {
       return yo`<label class="${data.children ? css.folder : css.file}"
         data-path="${data.path}"
         style="${isRoot ? 'font-weight:bold;' : ''}"
-        onload=${function (el) { adaptEnvironment(el, focus, hover, li) }}
-        onunload=${function (el) { unadaptEnvironment(el, focus, hover, li) }}
         onclick=${editModeOn}
         onkeydown=${editModeOff}
         onblur=${editModeOff}
@@ -117,6 +116,10 @@ function fileExplorer (appAPI, files) {
     }
   })
 
+  self.treeView.event.register('leafClick', function (key, data, label) {
+    self.events.trigger('focus', [key])
+  })
+
   self.treeView.event.register('nodeClick', function (path, childrenContainer) {
     if (!childrenContainer) return
     if (childrenContainer.style.display === 'none') {
@@ -141,18 +144,17 @@ function fileExplorer (appAPI, files) {
     return newList
   }
 
-  var deleteButton = yo`
-    <span class=${css.remove} onclick=${deletePath}>
-      <i class="fa fa-trash" aria-hidden="true"></i>
-    </span>
-  `
   appAPI.event.register('currentFileChanged', (newFile, explorer) => {
+    if (focusElement && newFile !== focusPath) {
+      focusElement.classList.remove(css.hasFocus)
+    }
     if (explorer === files) {
-      fileFocus(newFile)
-    } else {
-      var currentFile = appAPI.config.get('currentFile')
-      if (currentFile === newFile) return
-      unfocus(focusElement)
+      focusElement = self.treeView.labelAt(newFile)
+      if (focusElement) {
+        focusPath = newFile
+        focusElement.classList.add(css.hasFocus)
+        appAPI.config.set('currentFile', newFile)
+      }
     }
   })
   fileEvents.register('fileRemoved', fileRemoved)
@@ -187,59 +189,11 @@ function fileExplorer (appAPI, files) {
     }
   }
 
-  function focus (event) {
-    event.cancelBubble = true
-    var li = this
-    if (focusElement === li) return
-    unfocus(focusElement)
-    focusElement = li
-    focusElement.classList.toggle(css.hasFocus)
-    var label = getLabelFrom(li)
-    var filepath = label.dataset.path
-    var isFile = label.className.indexOf('file') === 0
-    if (isFile) self.events.trigger('focus', [filepath])
-  }
-
-  function unfocus (el) {
-    if (focusElement) focusElement.classList.toggle(css.hasFocus)
-    focusElement = null
-  }
-
-  function hover (event) {
-    var path = this.querySelector('label').dataset.path
-    if (path === self.files.type) return // can't delete the root node
-
-    if (event.type === 'mouseout') {
-      var exitedTo = event.toElement || event.relatedTarget
-      if (this.contains(exitedTo)) return
-      this.style.backgroundColor = ''
-      this.style.paddingRight = '19px'
-      return this.removeChild(deleteButton)
-    }
-    this.style.backgroundColor = styles.leftPanel.backgroundColor_FileExplorer
-    this.style.paddingRight = '0px'
-    this.appendChild(deleteButton)
-  }
-
   function getElement (path) {
     var label = self.element.querySelector(`label[data-path="${path}"]`)
     if (label) return getLiFrom(label)
   }
 
-  function deletePath (event) {
-    event.cancelBubble = true
-    var span = this
-    var li = span.parentElement.parentElement
-    var label = getLabelFrom(li)
-    var path = label.dataset.path
-    var isFolder = !!~label.className.indexOf('folder')
-    if (isFolder) path += '/'
-    modalDialogCustom.confirm(null, `Do you really want to delete "${path}" ?`, () => {
-      li.parentElement.removeChild(li)
-      removeSubtree(files, path, isFolder)
-    })
-  }
-
   function editModeOn (event) {
     if (self.files.readonly) return
     var label = this
@@ -292,15 +246,6 @@ function fileExplorer (appAPI, files) {
     }
   }
 
-  function fileFocus (path) {
-    var filepath = appAPI.config.get('currentFile')
-    if (filepath === path) return
-    var label = document.querySelector(`label[data-path="${path}"]`)
-    if (label) {
-      appAPI.config.set('currentFile', path)
-    }
-  }
-
   function fileRemoved (filepath) {
     var li = getElement(filepath)
     if (li) li.parentElement.removeChild(li)
@@ -336,58 +281,10 @@ function fileExplorer (appAPI, files) {
   }
 }
 
-/*
-  HELPER FUNCTIONS
-*/
-function adaptEnvironment (label, focus, hover) {
-  var li = getLiFrom(label) // @TODO: maybe this gets refactored?
-  li.style.position = 'relative'
-  var span = li.firstChild
-  // add focus
-  li.addEventListener('click', focus)
-  // add hover
-  span.classList.add(css.activeMode)
-  span.addEventListener('mouseover', hover)
-  span.addEventListener('mouseout', hover)
-}
-
-function unadaptEnvironment (label, focus, hover) {
-  var li = getLiFrom(label) // @TODO: maybe this gets refactored?
-  var span = li.firstChild
-  li.style.position = undefined
-  // remove focus
-  li.removeEventListener('click', focus)
-  // remove hover
-  span.classList.remove(css.activeMode)
-  span.removeEventListener('mouseover', hover)
-  span.removeEventListener('mouseout', hover)
-}
-
 function getLiFrom (label) {
   return label.parentElement.parentElement.parentElement
 }
 
-function getLabelFrom (li) {
-  return li.children[0].children[1].children[0]
-}
-
-function removeSubtree (files, path, isFolder) {
-  var parts = path.split('/')
-  var isFile = parts[parts.length - 1].length
-  var removePaths = isFile ? [path] : Object.keys(files.list()).filter(keep)
-  function keep (p) { return ~p.indexOf(path) }
-  removePaths.forEach(function (path) {
-    [...window.files.querySelectorAll('.file .name')].forEach(function (span) {
-      if (span.innerText === path) {
-        var li = span.parentElement
-        li.parentElement.removeChild(li) // delete tab
-      }
-    })
-    files.remove(path)
-  })
-  if (isFolder) files.remove(path)
-}
-
 fileExplorer.prototype.init = function () {
   this.container = yo`<div></div>`
   return this.container

From 26c38253116e2429a0b6c4b0621192b7cb59a727 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Fri, 2 Feb 2018 11:39:22 +0100
Subject: [PATCH 20/30] fix element

---
 src/app/files/file-explorer.js | 45 ++++++++++++++++++++--------------
 1 file changed, 26 insertions(+), 19 deletions(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 563f3c8da0..45d1e0b2c0 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -53,14 +53,12 @@ var css = csjs`
 `
 module.exports = fileExplorer
 
-var focusElement = null
-var focusPath = null
-
 function fileExplorer (appAPI, files) {
   var self = this
   this.events = new EventManager()
   this.files = files
-
+  this.focusElement = null
+  this.focusPath = null
   function remixdDialog () {
     return yo`<div>This file has been changed outside of Remix IDE.</div>`
   }
@@ -117,7 +115,18 @@ function fileExplorer (appAPI, files) {
   })
 
   self.treeView.event.register('leafClick', function (key, data, label) {
-    self.events.trigger('focus', [key])
+    if (self.focusElement) {
+      self.focusElement.classList.remove(css.hasFocus)
+      self.focusElement = null
+      self.focusPath = null
+    }
+    self.focusElement = self.treeView.labelAt(key)
+    if (self.focusElement) {
+      self.focusElement.classList.add(css.hasFocus)
+      self.focusPath = key
+      self.events.trigger('focus', [key])
+      appAPI.config.set('currentFile', key)
+    }
   })
 
   self.treeView.event.register('nodeClick', function (path, childrenContainer) {
@@ -130,8 +139,7 @@ function fileExplorer (appAPI, files) {
       if (error) console.error(error)
       if (!fileTree) return
       var newTree = normalize(path, fileTree)
-      newTree = self.treeView.renderProperties(newTree, false)
-      self.treeView.updateNode(path, newTree)
+      self.treeView.updateNodeFromJSON(path, newTree, false)
     })
   })
 
@@ -145,16 +153,10 @@ function fileExplorer (appAPI, files) {
   }
 
   appAPI.event.register('currentFileChanged', (newFile, explorer) => {
-    if (focusElement && newFile !== focusPath) {
-      focusElement.classList.remove(css.hasFocus)
-    }
-    if (explorer === files) {
-      focusElement = self.treeView.labelAt(newFile)
-      if (focusElement) {
-        focusPath = newFile
-        focusElement.classList.add(css.hasFocus)
-        appAPI.config.set('currentFile', newFile)
-      }
+    if (self.focusElement && explorer.type !== files.type && self.focusPath !== newFile) {
+      self.focusElement.classList.remove(css.hasFocus)
+      self.focusElement = null
+      self.focusPath = null
     }
   })
   fileEvents.register('fileRemoved', fileRemoved)
@@ -273,8 +275,13 @@ function fileExplorer (appAPI, files) {
           if (error) console.error(error)
           if (!fileTree) return
           fileTree = normalize(folderpath, fileTree)
-          var newTree = self.treeView.renderProperties(fileTree, false)
-          self.treeView.updateNode(folderpath, newTree)
+          self.treeView.updateNodeFromJSON(folderpath, fileTree, false)
+          self.focusElement = self.treeView.labelAt(self.focusPath)
+          // TODO: here we update the selected file (it applicable)
+          // cause we are refreshing the interface of the whole directory when there's a new file.
+          if (self.focusElement && !self.focusElement.classList.contains(css.hasFocus)) {
+            self.focusElement.classList.add(css.hasFocus)
+          }
         })
       }
     })

From 4ca1901407d3e623692c43354b08650a6721d17f Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Fri, 2 Feb 2018 14:55:06 +0100
Subject: [PATCH 21/30] add commnet and llittle cleaning

---
 src/app/files/basicReadOnlyExplorer.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/app/files/basicReadOnlyExplorer.js b/src/app/files/basicReadOnlyExplorer.js
index c9c00ac00e..a9178702ab 100644
--- a/src/app/files/basicReadOnlyExplorer.js
+++ b/src/app/files/basicReadOnlyExplorer.js
@@ -45,10 +45,10 @@ class BasicReadOnlyExplorer {
   }
 
   addReadOnly (path, content, rawPath) {
-    var unprefixedPath = this.removePrefix(path)
     try { // lazy try to format JSON
       content = JSON.stringify(JSON.parse(content), null, '\t')
     } catch (e) {}
+    // splitting off the path in a tree structure, the json tree is used in `resolveDirectory`
     var split = path
     var folder = false
     while (split.lastIndexOf('/') !== -1) {
@@ -61,9 +61,9 @@ class BasicReadOnlyExplorer {
       folder = true
     }
     this.paths[this.type][split] = { isDirectory: folder }
-    this.files[this.type + '/' + unprefixedPath] = content
+    this.files[path] = content
     this.normalizedNames[rawPath] = path
-    this.event.trigger('fileAdded', [this.type + '/' + unprefixedPath, true])
+    this.event.trigger('fileAdded', [path, true])
     return true
   }
 
@@ -87,6 +87,7 @@ class BasicReadOnlyExplorer {
     var self = this
     if (path[0] === '/') path = path.substring(1)
     if (!path) return callback(null, { [self.type]: { } })
+    // we just return the json tree populated by `addReadOnly`
     callback(null, this.paths[path])
   }
 

From 985e58f374c94b9bcf8c685dcf5d085212448414 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Fri, 2 Feb 2018 14:55:31 +0100
Subject: [PATCH 22/30] fix file-explorer "nodeClick"

---
 src/app/files/file-explorer.js | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
index 45d1e0b2c0..27fd5ff1fa 100755
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -131,15 +131,13 @@ function fileExplorer (appAPI, files) {
 
   self.treeView.event.register('nodeClick', function (path, childrenContainer) {
     if (!childrenContainer) return
-    if (childrenContainer.style.display === 'none') {
-      childrenContainer.innerHTML = ''
-      return
-    }
+    if (childrenContainer.style.display === 'none') return
+
     files.resolveDirectory(path, (error, fileTree) => {
       if (error) console.error(error)
       if (!fileTree) return
       var newTree = normalize(path, fileTree)
-      self.treeView.updateNodeFromJSON(path, newTree, false)
+      self.treeView.updateNodeFromJSON(path, newTree, true)
     })
   })
 
@@ -275,7 +273,7 @@ function fileExplorer (appAPI, files) {
           if (error) console.error(error)
           if (!fileTree) return
           fileTree = normalize(folderpath, fileTree)
-          self.treeView.updateNodeFromJSON(folderpath, fileTree, false)
+          self.treeView.updateNodeFromJSON(folderpath, fileTree, true)
           self.focusElement = self.treeView.labelAt(self.focusPath)
           // TODO: here we update the selected file (it applicable)
           // cause we are refreshing the interface of the whole directory when there's a new file.

From 1a9a559f874f199b6265a65ed615ac8a9ffad5d6 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Fri, 2 Feb 2018 15:12:21 +0100
Subject: [PATCH 23/30] move addFile to file-panel.js

---
 src/app/files/file-explorer.js | 24 ------------------------
 src/app/panels/file-panel.js   | 26 +++++++++++++++++++++++++-
 2 files changed, 25 insertions(+), 25 deletions(-)
 mode change 100755 => 100644 src/app/files/file-explorer.js

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
old mode 100755
new mode 100644
index 27fd5ff1fa..99f1ead8eb
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -165,30 +165,6 @@ function fileExplorer (appAPI, files) {
   var textUnderEdit = null
   var textInRename = false
 
-  self.api = {}
-  self.api.addFile = function addFile (file) {
-    function loadFile () {
-      var fileReader = new FileReader()
-      fileReader.onload = function (event) {
-        if (helper.checkSpecialChars(file.name)) {
-          modalDialogCustom.alert('Special characters are not allowed')
-          return
-        }
-        var success = files.set(name, event.target.result)
-        if (!success) modalDialogCustom.alert('Failed to create file ' + name)
-        else self.events.trigger('focus', [name])
-      }
-      fileReader.readAsText(file)
-    }
-
-    var name = files.type + '/' + file.name
-    if (!files.exists(name)) {
-      loadFile()
-    } else {
-      modalDialogCustom.confirm(null, `The file ${name} already exists! Would you like to overwrite it?`, () => { loadFile() })
-    }
-  }
-
   function getElement (path) {
     var label = self.element.querySelector(`label[data-path="${path}"]`)
     if (label) return getLiFrom(label)
diff --git a/src/app/panels/file-panel.js b/src/app/panels/file-panel.js
index 55a8f4e355..606c7607dc 100644
--- a/src/app/panels/file-panel.js
+++ b/src/app/panels/file-panel.js
@@ -1,3 +1,4 @@
+/* global FileReader */
 var async = require('async')
 var $ = require('jquery')
 var yo = require('yo-yo')
@@ -143,7 +144,30 @@ function filepanel (appAPI, filesProvider) {
     // the files module. Please ask the user here if they want to overwrite
     // a file and then just use `files.add`. The file explorer will
     // pick that up via the 'fileAdded' event from the files module.
-    ;[...this.files].forEach(fileExplorer.api.addFile)
+
+    ;[...this.files].forEach((file) => {
+      var files = fileExplorer.files
+      function loadFile () {
+        var fileReader = new FileReader()
+        fileReader.onload = function (event) {
+          if (helper.checkSpecialChars(file.name)) {
+            modalDialogCustom.alert('Special characters are not allowed')
+            return
+          }
+          var success = files.set(name, event.target.result)
+          if (!success) modalDialogCustom.alert('Failed to create file ' + name)
+          else self.events.trigger('focus', [name])
+        }
+        fileReader.readAsText(file)
+      }
+
+      var name = files.type + '/' + file.name
+      if (!files.exists(name)) {
+        loadFile()
+      } else {
+        modalDialogCustom.confirm(null, `The file ${name} already exists! Would you like to overwrite it?`, () => { loadFile() })
+      }
+    })
   }
 
   // ----------------- resizeable ui ---------------

From dabe16d17370501fe40b479a3312390d60a66311 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Fri, 2 Feb 2018 15:21:51 +0100
Subject: [PATCH 24/30] reorg function , add comment

---
 src/app/files/file-explorer.js | 112 +++++++++++++++------------------
 1 file changed, 52 insertions(+), 60 deletions(-)
 mode change 100644 => 100755 src/app/files/file-explorer.js

diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
old mode 100644
new mode 100755
index 99f1ead8eb..5c6bb27d07
--- a/src/app/files/file-explorer.js
+++ b/src/app/files/file-explorer.js
@@ -1,4 +1,3 @@
-/* global FileReader */
 var yo = require('yo-yo')
 var csjs = require('csjs-inject')
 var Treeview = require('remix-debugger').ui.TreeView
@@ -56,9 +55,14 @@ module.exports = fileExplorer
 function fileExplorer (appAPI, files) {
   var self = this
   this.events = new EventManager()
+  // file provider backend
   this.files = files
+  // element currently focused on
   this.focusElement = null
+  // path currently focused on
   this.focusPath = null
+
+  // warn if file changed outside of Remix
   function remixdDialog () {
     return yo`<div>This file has been changed outside of Remix IDE.</div>`
   }
@@ -80,8 +84,51 @@ function fileExplorer (appAPI, files) {
     }
   })
 
-  var fileEvents = files.event
+  // register to event of the file provider
+  files.event.register('fileRemoved', fileRemoved)
+  files.event.register('fileRenamed', fileRenamed)
+  files.event.register('fileRenamedError', fileRenamedError)
+  files.event.register('fileAdded', fileAdded)
+
+  function fileRenamedError (error) {
+    modalDialogCustom.alert(error)
+  }
+
+  function fileAdded (filepath) {
+    self.ensureRoot(() => {
+      var folderpath = filepath.split('/').slice(0, -1).join('/')
 
+      var currentTree = self.treeView.nodeAt(folderpath)
+      if (currentTree && self.treeView.isExpanded(folderpath)) {
+        self.files.resolveDirectory(folderpath, (error, fileTree) => {
+          if (error) console.error(error)
+          if (!fileTree) return
+          fileTree = normalize(folderpath, fileTree)
+          self.treeView.updateNodeFromJSON(folderpath, fileTree, true)
+          self.focusElement = self.treeView.labelAt(self.focusPath)
+          // TODO: here we update the selected file (it applicable)
+          // cause we are refreshing the interface of the whole directory when there's a new file.
+          if (self.focusElement && !self.focusElement.classList.contains(css.hasFocus)) {
+            self.focusElement.classList.add(css.hasFocus)
+          }
+        })
+      }
+    })
+  }
+
+  function fileRemoved (filepath) {
+    var label = this.treeView.labelAt(filepath)
+    if (label && label.parentElement) {
+      label.parentElement.removeChild(label)
+    }
+  }
+
+  function fileRenamed (oldName, newName, isFolder) {
+    fileRemoved(oldName)
+    fileAdded(newName)
+  }
+
+  // make interface and register to nodeClick, leafClick
   self.treeView = new Treeview({
     extractData: function extractData (value, tree, key) {
       var newValue = {}
@@ -150,6 +197,7 @@ function fileExplorer (appAPI, files) {
     return newList
   }
 
+  // register to main app, trigger when the current file in the editor changed
   appAPI.event.register('currentFileChanged', (newFile, explorer) => {
     if (self.focusElement && explorer.type !== files.type && self.focusPath !== newFile) {
       self.focusElement.classList.remove(css.hasFocus)
@@ -157,23 +205,14 @@ function fileExplorer (appAPI, files) {
       self.focusPath = null
     }
   })
-  fileEvents.register('fileRemoved', fileRemoved)
-  fileEvents.register('fileRenamed', fileRenamed)
-  fileEvents.register('fileRenamedError', fileRenamedError)
-  fileEvents.register('fileAdded', fileAdded)
 
   var textUnderEdit = null
   var textInRename = false
 
-  function getElement (path) {
-    var label = self.element.querySelector(`label[data-path="${path}"]`)
-    if (label) return getLiFrom(label)
-  }
-
   function editModeOn (event) {
     if (self.files.readonly) return
     var label = this
-    var li = getLiFrom(label)
+    var li = label.parentElement.parentElement.parentElement
     var classes = li.className
     if (~classes.indexOf('hasFocus') && !label.getAttribute('contenteditable') && label.getAttribute('data-path') !== self.files.type) {
       textUnderEdit = label.innerText
@@ -204,66 +243,19 @@ function fileExplorer (appAPI, files) {
       }
     }
 
-    function cancelRename () {
-      label.innerText = textUnderEdit
-    }
-
     if (event.which === 13) event.preventDefault()
     if (!textInRename && (event.type === 'blur' || event.which === 27 || event.which === 13) && label.getAttribute('contenteditable')) {
       textInRename = true
       var isFolder = label.className.indexOf('folder') !== -1
       var save = textUnderEdit !== label.innerText
       if (save) {
-        modalDialogCustom.confirm(null, 'Do you want to rename?', () => { rename() }, () => { cancelRename() })
+        modalDialogCustom.confirm(null, 'Do you want to rename?', () => { rename() }, () => { label.innerText = textUnderEdit })
       }
       label.removeAttribute('contenteditable')
       label.classList.remove(css.rename)
       textInRename = false
     }
   }
-
-  function fileRemoved (filepath) {
-    var li = getElement(filepath)
-    if (li) li.parentElement.removeChild(li)
-  }
-
-  function fileRenamed (oldName, newName, isFolder) {
-    var li = getElement(oldName)
-    if (li) {
-      li.parentElement.removeChild(li)
-      fileAdded(newName)
-    }
-  }
-
-  function fileRenamedError (error) {
-    modalDialogCustom.alert(error)
-  }
-
-  function fileAdded (filepath) {
-    self.ensureRoot(() => {
-      var folderpath = filepath.split('/').slice(0, -1).join('/')
-
-      var currentTree = self.treeView.nodeAt(folderpath)
-      if (currentTree && self.treeView.isExpanded(folderpath)) {
-        self.files.resolveDirectory(folderpath, (error, fileTree) => {
-          if (error) console.error(error)
-          if (!fileTree) return
-          fileTree = normalize(folderpath, fileTree)
-          self.treeView.updateNodeFromJSON(folderpath, fileTree, true)
-          self.focusElement = self.treeView.labelAt(self.focusPath)
-          // TODO: here we update the selected file (it applicable)
-          // cause we are refreshing the interface of the whole directory when there's a new file.
-          if (self.focusElement && !self.focusElement.classList.contains(css.hasFocus)) {
-            self.focusElement.classList.add(css.hasFocus)
-          }
-        })
-      }
-    })
-  }
-}
-
-function getLiFrom (label) {
-  return label.parentElement.parentElement.parentElement
 }
 
 fileExplorer.prototype.init = function () {

From 581bb7fe06d98f59444dc936fba20d2b351de21e Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 7 Feb 2018 11:42:53 +0100
Subject: [PATCH 25/30] fix first load

---
 src/app.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/app.js b/src/app.js
index b7fb76bc6b..0cff818aac 100644
--- a/src/app.js
+++ b/src/app.js
@@ -515,9 +515,11 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
   filePanel.event.register('resize', delta => self._adjustLayout('left', delta))
 
   var previouslyOpenedFile = config.get('currentFile')
-  filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
-    if (!error && content) fileManager.switchFile(previouslyOpenedFile)
-  })
+  if (previouslyOpenedFile) {
+    filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
+      if (!error && content) fileManager.switchFile(previouslyOpenedFile)
+    })
+  }
 
   // ----------------- Renderer -----------------
   var rendererAPI = {

From 3a94e484d7dfbba489d3045055f5f9d95e8ce75a Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 7 Feb 2018 12:13:04 +0100
Subject: [PATCH 26/30] fic build

---
 package.json | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package.json b/package.json
index 2b2d65b988..42060a51a2 100644
--- a/package.json
+++ b/package.json
@@ -139,8 +139,8 @@
     ]
   },
   "scripts": {
-    "setupremix": "npm run pullremix; npm run linkremixcore; npm run linkremixlib; npm run linkremixsolidity; npm run linkremixdebugger;",
-    "pullremix": "mkdir remix; git clone https://github.com/ethereum/remix; cd ..",
+    "setupremix": "npm run pullremix && npm run linkremixcore && npm run linkremixlib && npm run linkremixsolidity && npm run linkremixdebugger;",
+    "pullremix": "git clone https://github.com/ethereum/remix",
     "linkremixcore": "cd node_modules && rm -rf remix-core && ln -s ../remix/remix-core remix-core && cd ..",
     "linkremixlib": "cd node_modules && rm -rf remix-lib && ln -s ../remix/remix-lib remix-lib && cd ..",
     "linkremixsolidity": "cd node_modules && rm -rf remix-solidity && ln -s ../remix/remix-solidity remix-solidity && cd ..",

From c783b153570aff0d309d0a3d108edddc3059c241 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 7 Feb 2018 13:56:26 +0100
Subject: [PATCH 27/30] fix switch file

---
 src/app/files/fileManager.js | 28 ++++------------------------
 1 file changed, 4 insertions(+), 24 deletions(-)

diff --git a/src/app/files/fileManager.js b/src/app/files/fileManager.js
index 33c99accc5..7f5ba17298 100644
--- a/src/app/files/fileManager.js
+++ b/src/app/files/fileManager.js
@@ -122,12 +122,11 @@ class FileManager {
   switchFile (file) {
     var self = this
     if (!file) {
-      self.opt.filesProviders['browser'].resolveDirectory('/', (error, filesTree) => {
+      self.opt.filesProviders['browser'].resolveDirectory('browser', (error, filesTree) => {
         if (error) console.error(error)
-        var fileList = Object.keys(flatten(filesTree))
+        var fileList = Object.keys(filesTree)
         if (fileList.length) {
-          file = fileList[0]
-          if (file) _switchFile(file)
+          _switchFile(fileList[0])
         } else {
           self.event.trigger('currentFileChanged', [])
         }
@@ -135,6 +134,7 @@ class FileManager {
     } else _switchFile(file)
     function _switchFile (file) {
       self.saveCurrentFile()
+      self.opt.config.set('currentFile', file)
       self.refreshTabs(file)
       self.fileProviderOf(file).get(file, (error, content) => {
         if (error) {
@@ -180,23 +180,3 @@ class FileManager {
 }
 
 module.exports = FileManager
-
-function flatten (tree) {
-  var flat = {}
-  var names = Object.keys(tree || {})
-  if (!names.length) return
-  else {
-    names.forEach(name => {
-      if ('/content' in tree[name]) flat[name] = false
-      else {
-        var subflat = flatten(tree[name])
-        if (!subflat) {
-          // empty folder
-        } else {
-          Object.keys(subflat).forEach(path => { flat[name + '/' + path] = false })
-        }
-      }
-    })
-    return flat
-  }
-}

From 3c2332200cbd1af3b260de134111e9fdf7cbab07 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 7 Feb 2018 13:57:35 +0100
Subject: [PATCH 28/30] readd switchfile

---
 src/app.js | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/app.js b/src/app.js
index 0cff818aac..aa0f619cd5 100644
--- a/src/app.js
+++ b/src/app.js
@@ -519,6 +519,8 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
     filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
       if (!error && content) fileManager.switchFile(previouslyOpenedFile)
     })
+  } else {
+    fileManager.switchFile()
   }
 
   // ----------------- Renderer -----------------

From 6614eaff4083e8e1062586f01fbbfa1c87268019 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 7 Feb 2018 14:04:41 +0100
Subject: [PATCH 29/30] fix switchFile

---
 src/app/files/fileManager.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/app/files/fileManager.js b/src/app/files/fileManager.js
index 7f5ba17298..976bbda632 100644
--- a/src/app/files/fileManager.js
+++ b/src/app/files/fileManager.js
@@ -121,17 +121,19 @@ class FileManager {
 
   switchFile (file) {
     var self = this
-    if (!file) {
-      self.opt.filesProviders['browser'].resolveDirectory('browser', (error, filesTree) => {
+    if (file) return _switchFile(file)
+    else {
+      var browserProvider = self.opt.filesProviders['browser']
+      browserProvider.resolveDirectory('browser', (error, filesTree) => {
         if (error) console.error(error)
         var fileList = Object.keys(filesTree)
         if (fileList.length) {
-          _switchFile(fileList[0])
+          _switchFile(browserProvider.type + '/' + fileList[0])
         } else {
           self.event.trigger('currentFileChanged', [])
         }
       })
-    } else _switchFile(file)
+    }
     function _switchFile (file) {
       self.saveCurrentFile()
       self.opt.config.set('currentFile', file)

From 2af41b895bd3c207d2199a3532e15715c87a5864 Mon Sep 17 00:00:00 2001
From: yann300 <yann.levreau@gmail.com>
Date: Wed, 7 Feb 2018 14:33:31 +0100
Subject: [PATCH 30/30] rollback previouslyOpenedFile opening behavio

---
 src/app.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/app.js b/src/app.js
index aa0f619cd5..ab8d2f04c3 100644
--- a/src/app.js
+++ b/src/app.js
@@ -517,7 +517,11 @@ Please make a backup of your contracts and start using http://remix.ethereum.org
   var previouslyOpenedFile = config.get('currentFile')
   if (previouslyOpenedFile) {
     filesProviders['browser'].get(previouslyOpenedFile, (error, content) => {
-      if (!error && content) fileManager.switchFile(previouslyOpenedFile)
+      if (!error && content) {
+        fileManager.switchFile(previouslyOpenedFile)
+      } else {
+        fileManager.switchFile()
+      }
     })
   } else {
     fileManager.switchFile()