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

Add create folder icon #88825

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Conversation

adamscott
Copy link
Member

@adamscott adamscott commented Feb 25, 2024

Adds the missing "create folder" icon and replaces the "Create folder" text buttons of the FileDialog, the EditorFileDialog, and the ProjectDialog to an icon button.

New icon (click for better preview): FolderCreate

I opened the folder icon in Inkscape (Folder editor/icons/Folder.svg), traced the margin around the plus sign from the new file icon (New editor/icons/New.svg), did a difference operation on the folder with the trace, exported it to SVG, then I optimized it with SVGOMG.

Before After
Capture d’écran du 2024-02-25 11-56-08 Capture d’écran du 2024-02-25 11-53-59
Capture d’écran du 2024-02-25 11-56-30 Capture d’écran du 2024-02-26 09-09-02

As you can see, the top bar occupies now less real estate and it makes the file dialogs more intuitive to use.

Relates to #88827

Edits

  • 2024-02-26
    • Reverted the "Create Project" dialog edits.

Copy link
Member

@ajreckof ajreckof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it!
icon is clear and takes less place. Moreover it goes better with other button next to it that are icons only.

scene/gui/file_dialog.cpp Show resolved Hide resolved
Copy link
Contributor

@Mickeon Mickeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Desirable it would seem like.
Although I would keep both the name AND the icon for the Create New Project popup.

editor/gui/editor_file_dialog.cpp Show resolved Hide resolved
Copy link
Contributor

@MewPurPur MewPurPur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, I agree with Micky.

editor/icons/FolderCreate.svg Outdated Show resolved Hide resolved
scene/theme/icons/folder_create.svg Outdated Show resolved Hide resolved
@adamscott adamscott force-pushed the add-create-folder-icon branch 3 times, most recently from 8d95450 to c94d12d Compare February 25, 2024 18:35
@adamscott
Copy link
Member Author

Desirable it would seem like. Although I would keep both the name AND the icon for the Create New Project popup.

I don't agree. The button would be even larger than usual. And there's a tooltip if the user doesn't know what the icon does.

No objection in modifying the icon tooltip though.

@MewPurPur
Copy link
Contributor

MewPurPur commented Feb 25, 2024

Even with the current UI, there is enough space for many more buttons without getting in the way of project names of realistic size. Turning text into simple buttons is a compromise used to simplify complex UI and get more space, neither of which is a problem here. The compromise is that it reduces discoverability and clarity

@adamscott
Copy link
Member Author

Even with the current UI, there is enough space for many more buttons without getting in the way of project names of realistic size. Turning text into simple buttons is a compromise used to simplify complex UI and get more space, neither of which is a problem here. The compromise is that it reduces discoverability and clarity

Again, I don't agree. Either we keep the text only button (which is quite ugly), or we set the icon only button. Especially since icon only buttons are flat and text buttons aren't. Why having redundancy for such a small action?

No, icons aren't just to simplify complex UI and get more space. It's about UX. I think the button as it is is way too verbose for it's own good.

There's a reason shy road panels are using pictograms instead of text.

@adamscott
Copy link
Member Author

As discussed on the Godot Developer Chat:

I think the problem here is the fact that "Create folder" doesn't belong here.
What does it have to do with "Project Name"?

@KoBeWi then pointed out #56420

@akien-mga
Copy link
Member

I'm also concerned about this change to the Create Project dialog.

The current UX is bad, and creating a folder is almost always required. Otherwise, it's very easy to inadvertently create a project in a folder with pre-existing contents (e.g. your home/documents folder), leading to importing all your personal images, etc. and littering things with .import files and hidden folders.

So I wouldn't make this button smaller before we fix the current UX issues we have.

The changes to FileDialog are fine on the other hand.

@adamscott

This comment was marked as resolved.

@adamscott adamscott force-pushed the add-create-folder-icon branch from c94d12d to fd36662 Compare February 26, 2024 14:04
@adamscott
Copy link
Member Author

I'll remove the icon replacement for the Create Project UI.

Removed!

Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally, it works as expected.

@akien-mga akien-mga merged commit 4a49373 into godotengine:master Feb 27, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@adamscott
Copy link
Member Author

adamscott commented Mar 6, 2024

@Mickeon @MewPurPur @akien-mga Seems that I was wrong and you were right. My friend, who is a UX designer, sent me this article on icon usability from the Nielsen Norman Group (UX research group):

To help overcome the ambiguity that almost all icons face, a text label must be present alongside an icon to clarify its meaning in that particular context. (And even if you’re using a standard icon, it’s often safer to include a label, especially if you slightly altered the icon to match your aesthetic preferences or constraints.)

[...] stripping these icons of text labels renders them completely meaningless and is counterproductive to the goal of providing easy access to content.

So yeah, maybe we should add the folder icon next to the "Create Folder" button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants