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

Config: Create config directory if it was deleted while runtime #126

Merged
merged 2 commits into from
Oct 25, 2023

Conversation

tetektoza
Copy link
Contributor

@tetektoza tetektoza commented Oct 25, 2023

In few previous commit a fix for incorrect directory if tool was installed by dpkg was added, now this patch adds on top of that - if users have deleted files, we probably don't want to screw their directory wherever they ran D1GraphicsTool in by saving config file upon closing the app here, so it creates the path once again and writes config here.

Plus in a separate commit there's been change that if the user opens app for the first time - it will open his home directory by default instead of opening config's directory.

In few previous commit a fix for incorrect directory if tool was
installed by dpkg was added, now this patch adds on top of that - if
users have deleted files, we probably don't want to screw their
directory wherever they ran D1GraphicsTool in by saving config file upon
closing the app here, so it creates the path once again and writes
config here.
Previously, if user didn't have config files (which is (most of the
times) equivalent to running the app for the first time), then by
default next file that he wanted to open would be opened in the config
directory. This oneliner changes it, opening it in home directory by
default.
@AJenbo
Copy link
Member

AJenbo commented Oct 25, 2023

Nice with the home as default

@AJenbo AJenbo merged commit 49cfe20 into diasurgical:master Oct 25, 2023
5 checks passed
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