Skip to content

Add CollectHelper #162

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

Merged
merged 4 commits into from
Feb 26, 2019
Merged

Conversation

BzKevin
Copy link
Contributor

@BzKevin BzKevin commented Feb 13, 2019

Adds plugin CollectHelper which gives the user a few helpful buttons to change collections based on selected image.

Reworded the button text to better match with the rest of darktable's buttons
@wpferguson
Copy link
Member

@BzKevin I was thinking of writing a compare function similar to lightoom's using the collect filter to do it. I figured that I was going to have to play with collect to figure out how it worked, but you did it for me. Thanks

@BzKevin
Copy link
Contributor Author

BzKevin commented Feb 15, 2019

@wpferguson Awesome, if it is similar enough in content then maybe adding it in to this same script makes sense? Certainly it wouldn't be hard to duplicate the functionality I've already implemented for the other collect parameters, like filmstrips, day, place, etc.

But if you're thinking of making a module with checkboxes for each parameter and then filtering on the selected properties with the selected image's value that my be better served in a new script. Which would be an awesome script by the way.

Updated variable definitions with 'local' designation. Doing so slightly broke the script so I modified the function calls a tad to pass variables in and out as needed for "CollectOnAll" function to work.
Add translate-ability. Also adds a check that the selected image has an active color label and displays a message if it does not
@wpferguson
Copy link
Member

Looks good.

@wpferguson wpferguson merged commit e5537fd into darktable-org:master Feb 26, 2019
@BzKevin BzKevin deleted the BzKevin-CollectHelper branch March 4, 2019 17:13
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.

3 participants