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

[Feature] Button to merge unique existing tag list with scene scraped tag results #1599

Open
echo6ix opened this issue Jul 25, 2021 · 2 comments
Labels
feature Pull requests that add a new feature

Comments

@echo6ix
Copy link
Contributor

echo6ix commented Jul 25, 2021

Is your feature request related to a problem? Please describe.
When scraping scenes with existing tags if the user wants to incorporate any existing tags into the scraped scene tag results their only option is to manually input each of their existing tags into the scraped tag list.

Describe the solution you'd like
Add a merge button, such as a right arrow ➡. The placement would ideally be in the existing tags input group, below the "x" button.

When the merge button is clicked any existing tags that do not exist in the scraped scene tag results are immediately added to the scraped scene tag results input group.

Additional context
Add a tool-tip on the merge button indicating it's purpose, "Merge existing tags with scraped scene tag results"

@gitgiggety
Copy link
Contributor

This would be a nice feature indeed. Shouldn't be too hard to add. But styling / UI wise it's a different story. The X you're mentioning is either an X or a V to select if you want to use the existing or scraped values, and it thus is a big button which you can click. So adding an additional button there isn't really possible. Ideally the button would be placed in between the fields, but that would require more space and on mobile the space is already scarce. What might be some middle ground though is place the button on the row with the label (so in this case "Tags"). There is a lot of room there.

@WithoutPants do you have any opinion on this?

Oh, and same could obviously be added to other fields as well. Or at least performers comes to mind, as that's also a multi select. And not just for scenes but also for images, galleries and performers.

@WithoutPants
Copy link
Collaborator

I like this idea. I'd probably add it inline with the multi-select, or between the two fields.

@WithoutPants WithoutPants added the feature Pull requests that add a new feature label Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Pull requests that add a new feature
Projects
Status: No status
Development

No branches or pull requests

3 participants