search

geometer / FBReaderJ

Official FBReaderJ project repository
http://www.fbreader.org/FBReaderJ/
1.83k stars 802 forks source link

`<hr>` with CSS visibility:hidden + padding should show up as paragraph break but in FBReader it doesn't #473

Open ghost opened 6 years ago

ghost commented 6 years ago

I tried to introduce a paragraph break (empty blank line between two paragraphs with the following approach):

<p>Some text</p>
<hr class="paragraph_break"/>
<p>Following text</p>
.paragraph_break {visibility: hidden; height:0px;border-style:none;display:block;padding-bottom:0.5em;padding-top:0.5em;}

This works in all readers I have tested (Calibre, Kindle Previewer 3, Microsoft Edge built-in EPUB reader) just fine... but not in FBReader for Android. There, the <hr> just disappears without any indication anything is there, not even a hint of additional vertical blank space.

This seems like a major bug to me, because semantically, clearly there should be a visible separator there, and visibility:hidden is pretty well-defined (and it's not the same as display:none).

link2xt commented 6 years ago

See issue #30. <hr> is simply not supported. A major bug indeed, not fixed since 2012 for some reason.

ghost commented 6 years ago

Well, by now I have simply accepted my ebooks don't work correctly in FBreaderJ. They work fine literally everywhere else I tried, so at some point I simply stopped caring.