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

improvement suggestions for godot(3) gui #9931

Closed
puppetmaster- opened this issue Jul 27, 2017 · 9 comments
Closed

improvement suggestions for godot(3) gui #9931

puppetmaster- opened this issue Jul 27, 2017 · 9 comments

Comments

@puppetmaster-
Copy link

puppetmaster- commented Jul 27, 2017

Suggestions for improvement

  1. [context menu to load ressource] change click area after selecting (fix: Inspector: Right click on resource opens sub-menu. #10238, Right Click Resource to Load New Image, Material, etc... #9052)
    context_menu_to_load_ressource
    Suggestion: click area should be same size before and after selecting
  2. button too narrow to open the menu (fix Some theme and usability improvements #10006)
    image
  3. red (error) message when pressed on [new project] button (confusing as novice)
    image
    Suggestion: Give hint that default project path is empty and can be set on editor setting
  4. selecting script language don't change file extension(Path) automatic and give error message (fix Some theme and usability improvements #10006)
    script_file_extension
  5. tree point behave different
    tree_points
  6. editor settings don't show icon to reset to default (PR: Added the ability to revert to initial value in editor settings. #11564)
    reset_icon

I hope there are not to much for one issue. Thanks for all the cool new g3 improvement.

@djrm
Copy link
Contributor

djrm commented Jul 27, 2017

I agree with all of this.
About 1) i would prefer the menu to appear when the right mouse button is pressed (as contextual menu) if there is something currently loaded, let me take a look at most of this stuff.

@Zylann
Copy link
Contributor

Zylann commented Jul 27, 2017

I would also suggest a change of icon for the "help":
image
It looks like a very small magnifying glass and some kind of... chain, but looking closer it actually says "DOC", not a good choice for an icon opening reference docs IMO, maybe something more "single-object" and universal such as a question mark / book with bookmark / good old buoy would be better.

@27thLiz
Copy link
Contributor

27thLiz commented Jul 28, 2017

Regarding 3.: I think that ideally, we'd have a "frist start" dialog, asking about the default project location. (#8134)

@toger5
Copy link
Contributor

toger5 commented Aug 5, 2017

3 definitly has to be adressed, because choosijg a floder does not put the project intp the folde,instead it makes that folder the project.
With the auto importer this can have fatal reults when choosing the documents folder... All files will get imported/doplicated by godot
Maybe the logic ahould be changed to: location + project folder name.

@reduz
Copy link
Member

reduz commented Aug 25, 2017

Regarding #1 and #2, I think the option menu was made wider so in part this has improved. I need to make it so click happens on button release, so dragging the resource works again. About @djrm suggestion to use RMB, I think this is ok, but it's optional and should not remove the menu for discoverability reasons.

Regarding #3, feel free to fix it guys as you best see fit

#4 is a pain in the ass usability wise, this need to be fixed

#5 happens because for some reason the TreeItem is set as selectable, even though it has no benefit being selectable

#6 should not be too difficult...

@djrm
Copy link
Contributor

djrm commented Aug 25, 2017

i solved 4 a while back

@djrm
Copy link
Contributor

djrm commented Aug 31, 2017

i have a PR for 3, #10796

@djrm
Copy link
Contributor

djrm commented Sep 25, 2017

pr for 6 #11564

also about 5, it has to be discussed, the reason for it being selectable is that on non default actions, you can press the name to modify it, disabling selection causes avoids the renaming of the action. An option would be to add a button to modify the name, or use double click to update it.

@djrm
Copy link
Contributor

djrm commented Sep 25, 2017

i think this can be finally closed, maybe open another issue for 5, since it needs discussion.

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

8 participants