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

support FileDialog::Mode::OPEN_DIR for selecting a directory #4272

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

forrestjgq
Copy link
Contributor

@forrestjgq forrestjgq commented Nov 9, 2021

add OPEN_DIR besides OPEN and SAVE for FileDialog mode definition to select a directory in cpp as long as python.

Currently OPEN and SAVE mode of FileDialog select a file path like /path/to/file.json, there is no way to select a directory path like /path/to/dir/path. OPEN_DIR mode is used to select an existing directory path.


This change is Reviewable

@update-docs
Copy link

update-docs bot commented Nov 9, 2021

Thanks for submitting this pull request! The maintainers of this repository would appreciate if you could update the CHANGELOG.md based on your changes.

@errissa errissa self-requested a review November 11, 2021 02:01
@errissa
Copy link
Collaborator

errissa commented Nov 11, 2021

@forrestjgq Would you please comment on what the use case is for this PR?

@forrestjgq
Copy link
Contributor Author

Currently OPEN and SAVE mode of FileDialog select a file path like /path/to/file.json, there is no way to select a directory path like /path/to/dir/path. OPEN_DIR mode is used to select an existing directory path.

@yxlao yxlao added the status / to merge Looks good, merge after minor updates. label Nov 15, 2021
Copy link
Collaborator

@errissa errissa left a comment

Choose a reason for hiding this comment

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

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @forrestjgq)

@errissa errissa merged commit 1c747f6 into isl-org:master Nov 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status / to merge Looks good, merge after minor updates.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants