-
Notifications
You must be signed in to change notification settings - Fork 303
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
Several small fixes and improvements #48
Conversation
This makes the shortcuts work universally without having to add them to the model. Also fixes the problem with shortcuts not working after loading another configuration file.
The changeLabel event triggers when new labels are loaded. This would sometimes create an extra label item because the previous QListWidgetItem would be made the current, retaining an item scheduled for deletion.
Fix CI on Linux
I'm currently fixing the Appveyor CI. Seems that the latest version of |
CI is passing again! |
Thank you for your contribution, I will release if you have no other features to add ? |
There are two things that I would like to add in another PR: 'Save all masks' and keyboard shortcuts to select the next/previous file in the folder, but don't let me stop you from releasing another version. I'm ok either way! |
Thanks for making this tool! Comes in really handy!
While using it, I stumbled on some small issues, so here is a pull request which should fix those.