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

Added an opt-in solution for issue #27 and made issue #9 significantly easier. #51

Merged
merged 1 commit into from
Dec 19, 2012

Conversation

jmclocklin
Copy link

This changeset solves two problems by adding a new setting that allows
users to opt-in to use the "advanced" output file selector. This new
selector is simply an OpenFileDialog instead of a terrible
FolderBrowserDialog. The two problems that this new dialog solves are:
(1) A custom file name can be added as requested by issue #27, and (2)
It is much easier to edit the output folder with the OpenFileDialog
because you can type/paste a file path (which helps a bit with issue
#9).

NewStuff

This changeset solves two problems by adding a new setting that allows
users to opt-in to use the "advanced" output file selector.  This new
selector is simply an OpenFileDialog instead of a terrible
FolderBrowserDialog.   The two problems that this new dialog solves are:
(1) A custom file name can be added as requested by issue marklagendijk#27, and (2)
It is much easier to edit the output folder with the OpenFileDialog
because you can type/paste a file path (which helps a bit with issue
marklagendijk#9).
@marklagendijk
Copy link
Owner

@jmclocklin thanks!
I will look into merging your pull request somewehere this week.I have to merge it with my changes (I build in a decent folder dialog).

marklagendijk pushed a commit that referenced this pull request Dec 19, 2012
Added an opt-in solution for issue #27 and made issue #9 significantly easier.
@marklagendijk marklagendijk merged commit 647720e into marklagendijk:master Dec 19, 2012
@marklagendijk
Copy link
Owner

@jmclocklin tnx! I will have a look to see whether the opt-in is necessary, or whether it should just be changed to 'select output file'.

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

Successfully merging this pull request may close these issues.

2 participants