-
Notifications
You must be signed in to change notification settings - Fork 117
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
Fix multi-action dialogs #217
Conversation
double check you committed everything and tested it thouroughly...
|
Yeah, don't know why, but I must have forgotten to check in the file. I have never programmed python before, tbh. So I hope it works now. |
running with this PR leads to
seems like there is no clean error handling implemented for cases w/o wx. can you reorder the imports and softfail with some more meaningful msg? ... and obviously you need to tweak your code to pass the tests PS: can you elaborate on what exact problem this PR fixes? feel free to report an issue on that... THX |
Please excuse me for making so much work. I now try to deal with the errors regarding the missing |
@the-jesus thank you for your work and your explanations |
Haven't used inkscape-silhouette in a while (haven't happened to need to cut anything intricate) but would be happy to give this a spin. However, I am busy through Thanksgiving weekend -- can it wait until the following week? |
Hi, how have you been? |
This fixed the problem with the confirmation dialog of the multi-action dialog. Also fixing that presets couldn't be deleted.