-removed "rename" from folder's right click.

-added padding to context-menu
ethereum · Sep 26, 2019 · 9a74c70 · 9a74c70
1 parent 8143b8e
commit 9a74c70
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/src/app/files/file-explorer.js b/src/app/files/file-explorer.js
@@ -206,10 +206,14 @@ function fileExplorer (localRegistry, files, menuItems) {
         )
       }
     } else {
-      actions['Rename'] = () => {
-        if (self.files.isReadOnly(key)) { return tooltip('cannot rename folder. ' + self.files.type + ' is a read only explorer') }
-        var name = label.querySelector('span[data-path="' + key + '"]')
-        if (name) editModeOn(name)
+      // extracts first two folders
+      const firstTwoFolders = key.substring(0, key.indexOf('/', key.indexOf('/', 1) + 1))
+      if (!provider.isExternalFolder(firstTwoFolders)) {
+        actions['Rename'] = () => {
+          if (self.files.isReadOnly(key)) { return tooltip('cannot rename folder. ' + self.files.type + ' is a read only explorer') }
+          var name = label.querySelector('span[data-path="' + key + '"]')
+          if (name) editModeOn(name)
+        }
       }
     }
     actions['Delete'] = () => {

diff --git a/src/app/ui/contextMenu.js b/src/app/ui/contextMenu.js
@@ -45,7 +45,7 @@ module.exports = (event, items) => {
     current.onclick = () => { hide(null, true); items[item]() }
     return current
   })
-  var container = yo`<div class="${css.container} bg-light"><ul id='menuitems'>${menu}</ul></div>`
+  var container = yo`<div class="p-1 ${css.container} bg-light"><ul id='menuitems'>${menu}</ul></div>`
   container.style.left = event.pageX + 'px'
   container.style.top = event.pageY + 'px'
   container.style.display = 'block'