Thread: ePUB comic test
View Single Post
Old 03-17-2010, 09:09 AM   #14
charleski
Wizard
charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.charleski ought to be getting tired of karma fortunes by now.
 
Posts: 1,196
Karma: 1281258
Join Date: Sep 2009
Device: PRS-505
Looks good.
You might want to wrap the images in svg so that they scale and keep their aspect ratio whatever the aspect of the reader. Since your comic pages are all 449x649, try replacing the code of the body with
<svg version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="100%" height="100%" viewBox="0 0 449 649" preserveAspectRatio="xMidYMid meet" >
<image width="449" height="649" xlink:href="images/h01.jpg" />
</svg>
(Obviously change the 'h01.jpg' to point to the right image in each case.)

There are a few things you need to tidy up:
1) Change the encoding directive in your xhtml files (it's the first line) from
<?xml version="1.0" encoding="iso-8859-1"?>
to
<?xml version="1.0" encoding="utf-8"?>
2) Remove all the unneeded files (toc blank.ncx etc), and definitely remove page-template.xpgt
3) Your xhtml files need a <title> element in the head, even if it's just blank, so add
<title />
to the heads of them all.
4) Your xhtml files don't actually load the css file and define all their styling information individually. You certainly want to change that. Remove
<style type="text/css" title="override_css">
@page {padding: 0pt; margin:0pt}
body { text-align: center; padding:0pt; margin: 0pt; }
div { margin: 0pt; padding: 0pt; }
</style>
and replace it with
<link rel="stylesheet" href="stylesheet.css" type="text/css" />
You'll probably want to put the @page and body definitions in your stylesheet file.
5) You don't actually have a toc.ncx file. You need one, even if it only contains one navpoint. The media-type declared for it in the opf needs to be "application/x-dtbncx+xml"
6) The opf file needs to include all the jpegs in the manifest section.
charleski is offline   Reply With Quote