different epub interpretations of <br> ?

[Toxaris]

It would be nice if we could select a piece of text or a paragraph and select a style for it. Of course the styles would be the ones defined in the stylesheet(s). I believe it is on Valloric list, but at a low priority.

In this way a remove indent button would be obsolete.

[cybmole]

Originally Posted by Toxaris

It would be nice if we could select a piece of text or a paragraph and select a style for it. Of course the styles would be the ones defined in the stylesheet(s). I believe it is on Valloric list, but at a low priority.

In this way a remove indent button would be obsolete.

yes, that would be far better, and would also be similar to how you apply a style in Word.

[salty-horse]

I have a Sony PRS T1 with the "latest" firmware, and an ebook with <p><br/><p>'s that are not respected.
Replacing the <br/> with &nbsp; broke everything, and now there are only blank pages, and every page flip jumps to the next chapter.

I ended up using "..." as a workaround.

(I'm commenting on an old thread because I'll probably have this same problem in the future.)

[JSWolf]

Originally Posted by salty-horse

I have a Sony PRS T1 with the "latest" firmware, and an ebook with <p><br/><p>'s that are not respected.
Replacing the <br/> with &nbsp; broke everything, and now there are only blank pages, and every page flip jumps to the next chapter.

I ended up using "..." as a workaround.

(I'm commenting on an old thread because I'll probably have this same problem in the future.)

None of those solutions are optimal. You want CSS to add the 1em space.

.spacet {
  padding-top: 1em;
}
.spaceb {
  padding-bottom: 1em;
}

You can use CSS instead of lines that don't work or look good.

[salty-horse]

The amount of bugs in the old Digital Editions software always surprises me. (Originally, the ebook didn't load at all because it used unknown-to-it CSS3 "rem" units - instead of a fallback, it just crashes)

[JSWolf]

Originally Posted by salty-horse

The amount of bugs in the old Digital Editions software always surprises me. (Originally, the ebook didn't load at all because it used unknown-to-it CSS3 "rem" units - instead of a fallback, it just crashes)

A lot of programs don't deal with rem. So using it is not a good idea. em works.

[salty-horse]

Originally Posted by JSWolf

None of those solutions are optimal. You want CSS to add the 1em space.

Much better, thanks.
I replaced all:

<p class="CLASSNAME"><br/></p>

with

<div class="CLASSNAME"></div>

and changed the styles of those classes from "margin: 0 0 0 0; line-height: X" to "margin: X 0 X 0", and added a rule for "div.br" with a 1em margin.

[JSWolf]

Originally Posted by salty-horse

Much better, thanks.
I replaced all:

<p class="CLASSNAME"><br/></p>

with

<div class="CLASSNAME"></div>

and changed the styles of those classes from "margin: 0 0 0 0; line-height: X" to "margin: X 0 X 0", and added a rule for "div.br" with a 1em margin.

What I would use is the following so you don't have these extra lines that you don't really need.

<p class="spacet">Some Paragraph text.</p>

.spacet {
  padding-top: 1em;
}

The reason for the padding is that the 1em space doesn't get lost when the space is at the bottom or top of the screen.

[Turtle91]

That is certainly one way to do it - and it is better than using the <br/> tag... however, I think Wolfie's intent was to use the css on the paragraph that has text.

Instead of:

<p>This is a normal paragraph.</p>
<p>This is a normal paragraph.</p>
<p><br/></p>
<p>This is a normal paragraph with space above.</p>
<p>This is a normal paragraph.</p>

-OR-

<p>This is a normal paragraph.</p>
<p>This is a normal paragraph.</p>
<p class="space"></p>
<p>This is a normal paragraph with space above.</p>
<p>This is a normal paragraph.</p>

You could use this:

p.spacet {padding-top: 1em}

<p>This is a normal paragraph.</p>
<p>This is a normal paragraph.</p>
<p class="spacet">This is a normal paragraph with space above.</p>
<p>This is a normal paragraph.</p>

That provides clean HTML that can be consistently relied on.

[JSWolf]

Originally Posted by Turtle91

That is certainly one way to do it - and it is better than using the <br/> tag... however, I think Wolfie's intent was to use the css on the paragraph that has text.

Instead of:

<p>This is a normal paragraph.</p>
<p>This is a normal paragraph.</p>
<p><br/></p>
<p>This is a normal paragraph with space above.</p>
<p>This is a normal paragraph.</p>

-OR-

<p>This is a normal paragraph.</p>
<p>This is a normal paragraph.</p>
<p class="space"></p>
<p>This is a normal paragraph with space above.</p>
<p>This is a normal paragraph.</p>

You could use this:

p.spacet {padding-top: 1em}

<p>This is a normal paragraph.</p>
<p>This is a normal paragraph.</p>
<p class="spacet">This is a normal paragraph with space above.</p>
<p>This is a normal paragraph.</p>

That provides clean HTML that can be consistently relied on.

We are thinking alike. See my post before yours and you'll see.

[Turtle91]

Doh - ninja'd again!

[JSWolf]

Originally Posted by Turtle91

Doh - ninja'd again!

[salty-horse]

Editing the paragraphs above/below the spacer elements is more error-prone. I opted to make the least changes to the existing file.

[JSWolf]

Originally Posted by salty-horse

Editing the paragraphs above/below the spacer elements is more error-prone. I opted to make the least changes to the existing file.

No it's not more error prone. Keeping the rubbish for the space is more prone not to work. You want to do this in CSS. You want to get rid of those extra lines. That's how it will work best.

[salty-horse]

I'm just trying to get the ebook I purchased to work on my device, and I did. Improving the markup isn't my goal.