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

Tagged filters #66

Merged
merged 2 commits into from
Apr 19, 2018
Merged

Conversation

IonoclastBrigham
Copy link
Collaborator

Adds File > Filter > Tagged and File > Filter > Untagged menu options, with shortcut keys T and U respectively.

There's a bit of copypasta in there, so I'm not 100% sure how much of what I'm doing in the callbacks is strictly necessary or generally helpful. Could also abstract out the callback bodies with a higher-order function that returns a function, since they only differ in negating the filter predicate.

New filter submenu.
New untagged filter menu item.
Filters for images in collection which don't have database entries.
Commented-out stubs for `filter > tagged` and `filter > all`.
Moves Filter menu under File.
Adds proper Tagged filter on model of Untagged.
Gives up on the idea of an "All" filter, since filtering is destructive.
@lehitoskin lehitoskin merged commit 0231572 into lehitoskin:master Apr 19, 2018
@IonoclastBrigham IonoclastBrigham deleted the brig/tagged-filters branch April 21, 2018 20:01
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