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

feat(webui): prefill author selection when bulk editing books #1053

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

noaione
Copy link
Contributor

@noaione noaione commented Jan 22, 2023

The current implementation is that if we select multiple books like this:

  • One has some author added
  • Another is empty, or some of them filled

It will keep the whole thing empty.

This PR should make the selection prefilled with what has been added.

komga_author_prefill_bulk_demo.mp4

Some problems need addressing:
As an example, two books have the following filled in:

  • Book 1: Writer, Penciller
  • Book 2: (Same) Writer

Currently, the form will have Writer and Penciller prefilled. I don't know if you like this behaviour or prefer to leave the Penciller empty.

Changed to basically get the intersection of many books for each roles, so only authors that exist on multiple books will be prefilled on a roles.

@gotson gotson added the consider Not sure yet if this makes sense or not label Jun 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consider Not sure yet if this makes sense or not
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants