View Single Post
Old 06-25-2011, 02:25 AM   #164
ldolse
Wizard
ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.
 
Posts: 1,337
Karma: 123455
Join Date: Apr 2009
Location: Malaysia
Device: PRS-650, iPhone
Cool, that was about what I discovered too. I've passed an updated Quality Check over to kiwidude, I imagine it should go up relatively soon as long as he's happy with the changes.

The check compares all css - both css files and inline style tags against Calibre's preferences. The check is relaxed when checking inline styles, as Calibre only puts the top and bottom margins in @page and left and right go to the class assigned to body. Basically it only flags an inline margin as disagreeing with Calibre if something exceeds Calibre's prefs - if the in-line margins are less than Calibre's prefs it leaves them alone.

It also ignores titlepage.xhtml - I'll make this more robust at some later point and have it ignore anything marked explicitly as a cover in the manifest.

There's also a bug in modify ePub's margin rewrites, I'll be updating that shortly.
ldolse is offline   Reply With Quote