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

Add "toolbar" buttons for some list actions #1673

Merged
merged 7 commits into from
Aug 26, 2021

Conversation

gitgiggety
Copy link
Contributor

Personal pet peeve is having to click twice to start playback of selected scenes (once to open the actions menu and once more for "Play selected". This while the edit and delete buttons always appear right away after selecting a scene while I use those much less. So this fixes it my adding a play button next to the edit and delete buttons.

Also when adding images to a gallery for the first time I really had to search for how to do it. Obviously there is the Add tab, but after selecting some images I had to search how to actually save it, and apparently that's also in the ... menu. But not anymore, as now it's also available in the toolbar. And applied the same for removing images from the gallery. So both are now available as button next to the edit/delete buttons.

This works by adding an optional icon property to the "otherOperations" list. When an icon is set the button will be shown. Optionally buttonVariant can be provided as well, which is used to make "Remove from gallery" button red (/apply danger variant).

@gitgiggety
Copy link
Contributor Author

Forgot to mention. "Play selected" seems to be broken, also on my "production" environment, so not related to these changes.

@gitgiggety
Copy link
Contributor Author

Mentioned issue above with "Play selected" actually was a long standing bug when only having a single item selected. Fixed in #1674

@WithoutPants WithoutPants added this to the Version 0.9.0 milestone Aug 26, 2021
@WithoutPants WithoutPants added the improvement Something needed tweaking. label Aug 26, 2021
@WithoutPants WithoutPants merged commit 2e83405 into stashapp:develop Aug 26, 2021
@gitgiggety gitgiggety deleted the list-operation-buttons branch August 30, 2021 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Something needed tweaking.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants