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

[Enhancement]: Allow ebooks to be marked as a supplement #1664

Closed
divyangbw opened this issue Apr 11, 2023 · 9 comments
Closed

[Enhancement]: Allow ebooks to be marked as a supplement #1664

divyangbw opened this issue Apr 11, 2023 · 9 comments
Labels
enhancement New feature or request

Comments

@divyangbw
Copy link
Contributor

Describe the feature/enhancement

ATM, for a library item with an ebook attached to it, you'll simply see "Read". There's 2 problems:

  1. Sometimes audiobooks come with supplementary material in the format of ebooks. This is not the same thing as the actual ebook version of it.
  2. Users will assume "Read" implies reading the actual ebook version of the audiobook.

Proposal:

Add a library wide setting to mark ebooks as supplements. For now, that will simply change the text from "Read" to "Supplement".

This reduces confusion for something like my use case where I'll have a different library for the ebooks vs audiobooks.

I'm new to Vue so going to explore a bit before opening a PR (and of course see if there's better ideas). Here's what I have so far:

Screenshot 2023-04-10 at 11 08 51 PM

Screenshot 2023-04-10 at 11 10 19 PM

@divyangbw divyangbw added the enhancement New feature or request label Apr 11, 2023
@advplyr
Copy link
Owner

advplyr commented Apr 11, 2023

What about if we have an ebook that has supplementary material?
What about if we have multiple files that are supplementary?

One thing I had a while back was a read button in the library files table so you could have multiple ebook files and be able to read them.
image

Maybe we have some way to mark an ebook as supplementary?

There is another feature request open that comes down to the user doesn't want ebooks to show at all. If we had a library setting like "Audiobooks only" then we automatically treat any extra ebook files as supplementary. This would solve both cases.

Just throwing some thoughts out there

@divyangbw
Copy link
Contributor Author

divyangbw commented Apr 11, 2023

What about if we have an ebook that has supplementary material?

Yea, I considered that, and it took me down the path you described right after -- mark each individual file as supplementary. Problem is it becomes tricky since I wasn't sure how to consider something as a "main item". How do we update the UI to differentiate reading the main thing vs supp. Or what happens if some mp3 is tagged is supplement but in reality its a chapter. How does file scanning get affected by this. Etc. Lots of questions 😅

Don't have answers since I'm still in the process of understanding the source code, and don't want to blindly do something. Could limit it to just epub, pdf, mobi, etc first as well, and not worry about some of those questions above. I can definitely see this as an improvement to make in the future.

What about if we have multiple files that are supplementary?

How do we currently handle multiple files (say on an audiobook item)? From a quick look it seems it picks the first file from the list and shows that reader. So regardless of the changes here, wouldn't we need to add better support for multiple files (like your idea around read button on the files table) first? I see that as a different feature 🤷🏽‍♂️ One that could affect what is done here when complete though.

There is another feature request open that comes down to the user doesn't want ebooks to show at all. If we had a library setting like "Audiobooks only" then we automatically treat any extra ebook files as supplementary. This would solve both cases.

Hmm, so assuming we don't want a new type (instead of just "book"), I'm guessing you mean if there's no audio tracks, hide the item everywhere (including under home and series)? Yea, I could look into that. Can you link me the issue if you have it handy?

I'm curious if there's a need for hiding, but not marking attached as supplements. For example if I only want to show the item if the audiobook has been added, but the attached ebook is the text version of that book. This way one could start adding their ebooks while they wait on the audiobook.

@advplyr
Copy link
Owner

advplyr commented Apr 11, 2023

If there are multiple ebook files found then it will take the first one and create an EbookFile which gets stored on the Book object.

All the other ebook files just remain in the library files array on the LibraryItem. Basically any ebook file that is not an EbookFile attached to the book would be supplementary, if that makes sense. So, I don't think we would actually need a flag for it to be supplementary or not.

The ebook file that was actually created as an ebook could be marked as supplementary by just removing it from the Book. In that case maybe we would allow you to set an ebook file as the actual ebook if there isn't one already. That would solve the case where Abs marked the wrong ebook file as the ebook.

As I was typing this I realized it probably wouldn't make much sense so I think I'll have to put something together to show what I mean.

@divyangbw
Copy link
Contributor Author

Ah, I see. I think I'm getting the gist of it. I'll wait on working on this in the meantime then. Will continue to play with the code in the meantime to get a better understanding. Thanks :) Worst case, the change I had is simple and we can always fallback on that.

@KK4BitsPerSecond
Copy link

Support for supplements in the audiobook library would be nice.

I have many audiobooks that came with PDFs of things like world maps and illustrations that I would love if I could access from the book in the audiobook library.

@advplyr advplyr added the awaiting release Issue is resolved and will be in the next release label Jun 10, 2023
@advplyr
Copy link
Owner

advplyr commented Jun 10, 2023

Added in v2.2.23

There was a lot here so I put it in a guide https://www.audiobookshelf.org/guides/ebooks

@advplyr advplyr closed this as completed Jun 10, 2023
@advplyr advplyr removed the awaiting release Issue is resolved and will be in the next release label Jun 10, 2023
@skyzuma
Copy link

skyzuma commented Jun 11, 2023

perfect option !

the "problem" is, the entire book (audio / eBook)will display on deck but i can only play the audiobook (play button) ... maybe if is possible make 2 decks ? 1 for audiobook if played and 1 for eBook if read? or make playbutton twice (if played) like play audio book (play button) / read book (read button)?

edit: if i look into the book i see only one progress and i cant see for what the progress stands for

edit1: maybe its possible and logical make the progres info under the relevant button? if i played the audiobook > progress under the play button and if i read the book make progress info under the read button ... idk how it works in the mobile app ...

edit2: if i add an ebook (only), abs in reading the foldername (ebook) as title in the edit modal instead of the epub filename and the first part of the ebook file for the author ...

i managed my ebook in an seperate folder:

abook

  • author
    • series
      • Volume
        • files
      • ebook
        • files

edit2.1 ... ok, that was my fault (?) ... if an audiobook is there and i add an ebook, abs will add this without any problems (any name) ... but if i add an ebook without an audiobook (i have only the ebook and the audiobook comes later or its not available) i need to name the ebook like the dokumenation "Vol X - Volumename" > so now abs can read it correctly in the edit modal ...

edit:3: delete an ebook (any) > abs will delete this from the system ... not like the standard: delete from abs / delete from system too ... there is no choise ...

@skyzuma
Copy link

skyzuma commented Jun 15, 2023

idk how github handle (re)edits and give every edit an notification or only once on its create ... have u read all the edits ? @advplyr

@advplyr
Copy link
Owner

advplyr commented Jun 15, 2023

Yes you can't just remove the ebook from Abs only. If you want to remove the ebook then manually move it out of the audiobook folder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants