Include title in document about paperoni HOT 6 OPEN

Hello @gdetrez. I've added that in on the dev branch to just work by default. I hadn't planned on making a release until around late January 2022. Will you be able to compile a version from the dev branch just fine so you can use it? If not, then I can still publish a release so you can use it.

from paperoni.

Thanks @hipstermojo (and happy new year!)

Building the dev branch right now to test the new feature 😄

from paperoni.

Built and tested, and it works great! Thx!
I noticed you used the page title (the content of <title> in HTML) and not the first heading (usually <h1>) as the title, any particular reason?

from paperoni.

Hi @gdetrez . Happy new year to you too!
The reason for that is actually trying to remain consistent with the original mozilla readability JS version. It first checks for the title from the <meta> tags, the <title> tag and then goes looking for it in the header tags <h1> and <h2> tags.
If you're running Firefox, you can try opening the same article in reader view and you should see the same effect over there. If not, please send me the links because that might be a bug in paperoni instead.

Also note, the current commit has some unfinished work that will cause paperoni to panic/crash if your article can't be extracted. Please keep that in mind 😅

from paperoni.

Thanks for the explanation. That makes sense.
(dev paperoni hasn't crashed on me so far 🤞)

from paperoni.

Alright then. I'll keep this open until I merge all the new changes.

from paperoni.