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

Unify filter feature and improve space management for EditorFileDialog #98022

Merged

Conversation

vPumpking
Copy link
Contributor

@vPumpking vPumpking commented Oct 9, 2024

Move buttons related to the file view on the container above so that they take less horizontal space and make search bar similar to the one in FileDialog

This is part 2 of proposal godotengine/godot-proposals#9099
requested by @Calinou after #88673 (review)

see #97344 and #88673
replaces PR #97941 which was based on a forked branch instead of upstream

@vPumpking
Copy link
Contributor Author

vPumpking commented Oct 12, 2024

feature is ready

EDIT: but there is still that problem that was introduced before my PR: the makedir button is offscreen

@vPumpking vPumpking changed the title Improve space management for EditorFileDialog Unify filter function and improve space management for EditorFileDialog Oct 13, 2024
@Calinou
Copy link
Member

Calinou commented Oct 30, 2024

There are some merge conflicts, could you rebase on top of master so I can review this?

@vPumpking
Copy link
Contributor Author

sure, tomorrow that'll be done

@vPumpking vPumpking changed the title Unify filter function and improve space management for EditorFileDialog Unify filter feature and improve space management for EditorFileDialog Oct 30, 2024
@vPumpking vPumpking force-pushed the editor_file_dialog_filter_master branch 2 times, most recently from 85e5526 to dd6e8f7 Compare November 3, 2024 17:24
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. Code looks good to me.

While testing this PR, I found an additional usability issue: #98838

editor/gui/editor_file_dialog.cpp Outdated Show resolved Hide resolved
editor/gui/editor_file_dialog.cpp Outdated Show resolved Hide resolved
@vPumpking vPumpking force-pushed the editor_file_dialog_filter_master branch from dd6e8f7 to f33415a Compare November 22, 2024 17:54
editor/gui/editor_file_dialog.cpp Outdated Show resolved Hide resolved
Move buttons related to the file view on the container above so that they take less horizontal space and make search bar similar to the one in FileDialog
@vPumpking vPumpking force-pushed the editor_file_dialog_filter_master branch from f33415a to 0542bcc Compare November 22, 2024 17:59
@akien-mga akien-mga modified the milestones: 4.x, 4.4 Nov 28, 2024
@akien-mga akien-mga merged commit bfa38e4 into godotengine:master Nov 29, 2024
20 checks passed
@akien-mga
Copy link
Member

Thanks!

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.

4 participants