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

Overhaul custom library creation window #759

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

NotArme
Copy link
Contributor

@NotArme NotArme commented Feb 23, 2025

This PR attempts to fix all bugs outlined on Issue 727.

Changes

An editable line was added to make it easier to manually add a path or change the filename
When only a directory is chosen for the path, a file name will now be automatically created based on the chosen library name
"Ok" button is now disabled if library name is empty
The file dialog will now correctly ask for a path instead of an existing file
Library creation window layout and size was slightly modified to accomodate changes

image
image

TODO:

The button for folder selection still needs an icon

Will now ask for a path instead of a file
Will create a new .json file on the selected path
Custom libraries is now also saved on creation, not only on changes
Create library window will now allow for either manual input of directory, or choosing it through a button
File name will be automatically created based on library name if not specified
Window was slightly resized to accomodate changes
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.

1 participant