LNReader / lnreader

Light novel reader for Android.
https://lnreader.github.io
MIT License
1.8k stars 191 forks source link

Export EPUB: <br> missing on every exported EPUB #777

Open Palloxin opened 1 year ago

Palloxin commented 1 year ago

Steps to reproduce

  1. go to Panda Machine translations
  2. search Into the Creations and download chapter 1
  3. export the novel (so only chap 1) with js included(because i did so and i dont want to test if it appears when i dont include JS)
  4. check with an editor the extracted html of chap 1 (of the epub, see images below)

Expected behavior

IMG_20230908_153910

(This a simple screenshot from Lnreader reader of the chapter so i expect the epub to be reproduced the same as in Lnreader, aesthetically)

Actual behavior

IMG_20230908_160117

(Into an epub3 reader)

Looking into the html file u can see this IMG_20230908_155549 (so it is not the JS fault) Whatever there was there in the chapter when using lnreader, it is not present in the extracted epub.

EDIT: every <br> is missing

LNReader version

1.1.16

Android version

Android 8 miui 8

Device

Huawei p8 lite 2017

Other details

No js applied in the text of the image in Expected Behaviour.

Acknowledgements

KrisadaFantasy commented 9 months ago

Not just missing on exported EPUB, but probably in imported as well!

I made the EPUB file myself in Calibre and imported it to the app. Found that LNReader is not rendered <br/> properly so I come here to see if there's an existing issue. At first I just thought that the app has issue on rendering HTML or something similar, but @Palloxin screenshot show the downloaded text rendered <br/> correctly. So the issue is something remove <br/> during import/export EPUB process?

Edit: I edit comment because GitHub actually line break <br/> in the comment that I forgot to marked as code...