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

File Dialogs adjustments (v2) #13343

Closed

Conversation

Krakean
Copy link
Contributor

@Krakean Krakean commented Nov 27, 2017

Made adjustments to open/save file dialogs according to notes from Akien (see comment #13252 (comment)), plus a bit more:

  1. Added "go to parent folder" (^) button to Save a File dialog.
  2. Tree.cpp: "nothing_selected" signal has been re-made
  3. Fixed issue in Project Export dialog: MODE_SAVE_FILE wasn't set when you click "Export".

See commit message for all details.

1. Removed "..", instead you now will see "Select Current Folder" and "Select this Folder" buttons.
2. Added "go to parent folder" (^) button to Save a File dialog.
3. Tree.cpp: "nothing_selected" signal has been re-made (previous implementation, merged in godotengine#13308, wasn't optimal in context of performance)
4. Fixed issue in Project Export dialog: MODE_SAVE_FILE wasn't set when you click "Export".
5. Now you can deselect items by clicking on empty space in Open a Directory dialog.
@Krakean Krakean force-pushed the filedialogs_adjustments_v2 branch from 58cb059 to 3e0ce97 Compare November 27, 2017 17:15
@akien-mga
Copy link
Member

@akien-mga
Copy link
Member

Merged manually as ad351a2 to fix style. Good job, it works fine :)

@akien-mga akien-mga closed this Nov 27, 2017
@Krakean
Copy link
Contributor Author

Krakean commented Nov 27, 2017

@akien-mga will fix at morning, if this can't be merged manually due to this style issue.

@Krakean
Copy link
Contributor Author

Krakean commented Nov 27, 2017

Oh, already merged. Thanks :)

@Krakean Krakean deleted the filedialogs_adjustments_v2 branch November 29, 2017 10:46
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.

2 participants