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

Rearrange "Open In" menu items of FileSystem dock #90570

Merged
merged 1 commit into from
May 29, 2024

Conversation

timothyqiu
Copy link
Member

Closes godotengine/godot-proposals#9483

BeforeAfter
Directory

before-file

after-file

File

before-dir

after-dir

This PR:

  • Makes "Show in File Manager" the last one because it's the most frequently used one.
  • Makes the terminal one first because it operates on the containing folder, different from the other two.

The diff looks messy. It's actually just swapping those two groups of codes.

@timothyqiu timothyqiu requested a review from a team as a code owner April 12, 2024 09:12
@AThousandShips AThousandShips added this to the 4.x milestone Apr 12, 2024
@KoBeWi KoBeWi requested a review from a team April 12, 2024 15:47
@kitbdev
Copy link
Contributor

kitbdev commented Apr 12, 2024

Should also be done in _file_list_empty_clicked and _tree_empty_click.

@akien-mga
Copy link
Member

Makes "Show in File Manager" the last one because it's the most frequently used one.

Is that a well established UX pattern? I would expect the most frequently used one to be the first entry in a list, not the last.

@akien-mga akien-mga modified the milestones: 4.x, 4.3 Apr 16, 2024
@KoBeWi
Copy link
Member

KoBeWi commented Apr 16, 2024

It can be either first or last element, as they are most accessible (you don't have to search in the middle of the list).
And arguably, separators can be treated like "sub-lists", so the position was fine before.

I accidentally opened the terminal at least twice so far, while trying to open something in external program. If the least used option is supposed to be in the middle, for me that would be the terminal one.

@timothyqiu
Copy link
Member Author

Is that a well established UX pattern? I would expect the most frequently used one to be the first entry in a list, not the last.

Yes. The last item can be easily located too. It's the same reason Close Window button being the last entry on the title bar (it's the first entry for a macOS window as these buttons are left aligned there).

Although being the first entry after the separator, "Show in File Manager" is still in the middle of the context menu.

@akien-mga akien-mga merged commit d489d4a into godotengine:master May 29, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@timothyqiu timothyqiu deleted the at-last branch May 30, 2024 04:45
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