readium / readium-css

🌈 A set of reference stylesheets for EPUB Reading Systems, starting with Readium Mobile
https://readium.org/readium-css/
BSD 3-Clause "New" or "Revised" License
90 stars 20 forks source link

Margin bottom ignored #93

Open gregoriopellegrino opened 3 years ago

gregoriopellegrino commented 3 years ago

I'm submitting a bug report.

Short description of the issue/suggestion: I am working on an EPUB file and testing it with Readium Desktop. In two cases the margin-bottom rule is ignored.

Steps to reproduce the issue/enhancement:

  1. I can send with private email the EPUB file
  2. I due casi sono:

HTML:


<header id="chapter-1">
      <h1>Title</h1>
      <p class="subtitle" role="doc-subtitle">Subtitle</p>
</header>
<p class="p">Text...</p>

CSS:

h1 + p.subtitle {
  font-size: 1.5em; /* x-large */
  font-style: normal;
  font-weight: normal;
  text-align: center;
  margin-top: -1.5em;
  margin-bottom: 2em;
  font-family: sans-serif;
}

HTML:

<p class="autore"><strong>Name</strong> bla bla bla</p>
<p class="autore"><strong>Name</strong> bla bla bla</p>

CSS:

p.autore {
  font-size: 1em;
  text-indent: 0;
  text-align: justify;
  margin-top: 0;
  margin-bottom: 1em;
}

What is the expected behaviour? FIRST: image SECOND: image

What is the current behaviour? FIRST: image SECOND: image

Do you know which CSS modules (stylesheets) are impacted? No...

Please tell us about your environment:

JayPanoz commented 3 years ago

@gregoriopellegrino by any chance, do you have publishers’ styles disabled and paragraph margins set in advanced settings?

We discussed advanced settings during the engineering call last week and are aware that there are existing issues re. their management when they have not been set and we need a default.

gregoriopellegrino commented 3 years ago

I have the default settings:

image

NicolaCavalli commented 3 years ago

I experienced the same bug, actually I could not find a way to insert spacing before or after an element. Anyone has faced the same issue?