Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editor: FileSystem drawer UI improvements #12322

Closed
mhilbrunner opened this issue Oct 22, 2017 · 3 comments
Closed

Editor: FileSystem drawer UI improvements #12322

mhilbrunner opened this issue Oct 22, 2017 · 3 comments

Comments

@mhilbrunner
Copy link
Member

This issue is to collect (and discuss) suggestions for improvements of the FileSystem drawer UI, making it (hopefully) even more useful and intuitive.

filesystem_drawer

Suggested improvements:

  • Dragging a folder onto itself in the directory tree view at the top yields a big Cannot move a folder into itself error dialog (Screenshot). Suggestion: Remove this error and do nothing or just log it.
  • The context menu for both the top and the bottom part is only available when right clicking existing items. So you have to right click an existing folder or file to create a new folder. Suggestion: Add a default context menu when right clicking the background that allows creating folders and Show In File manager.
  • The Create New Resource dialog is currently only available via the inspector, not via the FileSystem drawer. Suggestion: Add New resource.. option to the context menu(s), next to New folder.
  • In addition to the above, add a new icon next to the list icon and search bar which allows to create new folders and resources.

What do you think? Other suggestions?

@ghost
Copy link

ghost commented Oct 23, 2017

Regarding dragging onto itself this should probably be tested for in the drag & drop logic and not accept the drag if it would move a folder into itself.

@akien-mga akien-mga changed the title [Feature request] Editor: FileSystem drawer UI improvements Editor: FileSystem drawer UI improvements Nov 1, 2017
@mhilbrunner
Copy link
Member Author

Thanks @MillionOstrich for working on this :)

@mhilbrunner
Copy link
Member Author

Seeing everything but the resource thing got implemented, I'll close this and open a separate issue for that later. Thanks for working on this, MillionOstrich!

@akien-mga akien-mga added this to the 3.0 milestone Jan 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants