Quote:
Originally Posted by JSWolf
I've figured out what is going on and why the bug exists.
the blank lines are <br /> and they are not being picked up and converted to a blank line. If you fix that, you'll be good to go. I looked at the expanded HTML and yes, it had <br /> for the blank lines.
So no, I do not need to test version 7. Just get a fixed version 8 or a version 9.
|
You are right about the <br /> issue. It surfaces when using the '--nopara' as the eBook Publisher doesn't seem to respond to it beside the <div> construct. I have seen this in past conversions I did before the 'mobi2imp' days.
I have a work-around fix that could be inserted after line 289 in 'mobi2imp.pl' (just after the <body> tag substitution):
Code:
if (defined $opt_nopara) {
$html =~ s/<br([^>])*><div/<BR \/><BR \/><div/g; #force <br /> to work better in ebook Publisher
}
This is better than just forcing two <br />'s everywhere (what I tried first and didn't like!) Further testing is required to ensure this doesn't 'break' something else...
I used this 'fix' to produce the attached .IMP version of 'The Heretic.prc'
Is this better?
-Nick
p.s. the links to Chapter 9, Chapter 10 and Chapter 22 don't get fixed by the mobi code in 'mobi2imp' so you may want to check the original .prc to see if it is working properly. Also, in Chapter 40, I noticed an extra line para break ('<br /><br /><div') where the original .prc has a '<br /><div'
which probably shouldn't be there