Conversation
|
@daveallie I tried refactoring it, let me know if it seemed okay to you. Edit: Since the refactor, I have an issue with inline aside footnotes. I'll need to compare the code that was working with the current one to find the problem with a clear head. |
This reverts commit ad7e9bd.
|
Hi @jlaunay, thanks for submitting this, I'm still getting around to reviewing it, don't worry, I haven't forgotten about it! |
|
Hello, the original version before the refactor (here) worked very well, but given the number of changes in the master branch (screen to activities etc.) and the merges that were done quickly, I cannot manage to refactor it properly. If someone can help, I would really appreciate it. I lack the skills and the time to get it done :/ |
|
Apologies, things have been moving pretty quickly, I am happy to help jump in here and help out. I should have some time in the next two days to sit down and parse through this PR to get it back up to speed. |
|
would love to see footnote support come soon, but no rush! plenty of books that are footnote free to read :) cheers! |
Summary
This PR adds full support for three types of EPUB footnotes: inline
<aside>elements, paragraph notes<p class="note">, and external footnote references. Users can now view footnotes directly from any page and navigate to their content seamlessly.What changes are included?