Created attachment 533927 [details] Patch to recognize <graphic> like <imagedata> <imagedata> is not the only way to include images in a document, there's also <graphic>. It would be nice to support <graphic> too. I attach a patch which tells publican to deal with it in the same way. Seems to work here. BTW I noticed something odd while reading that part of the code: my $max_width = $self->{publican}->param('max_image_width') || 10000; my $set_width = $node->attr('width') || 0; $set_width =~ s/[^\d]//g; Ok max_width is 444, and set_width is the custom width if any. if ( $set_width == 444 ) { $node->attr( 'width', undef ); $set_width = 0; } We drop the attribute if the user set a custom width of 444. I don't see why, it could make sense to force a width of 444 if the user changed its max_image_width. But why not, it does not hurt. # Do not override user settings if ($set_width == 0) { $max_width = $set_width if ( $set_width && $set_width < $max_width ); Here it gets messy... we test $set_width == 0 so we know it's zero and we know that the test "if ($set_width && ...)" will fail, so this line is never executed. Maybe you wanted that line in the previous test ($set_width == 444) so that the max_width is reduced accordingly to the user specified size.
Added patch, added extra rules to XmlClean to ensure short cut tagging applies when required. Applied fixe to branches/publican-2x and trunk. Committed revision 1950. FYI the max_width code no longer exists in publican-2x or trunk, since we are moving away from FOP. It only existed to overcome FOP limitations.
oops, removed change from trunk as Publican 3 will use DocBook 5 and graphic has been discarded in DocBook 5. Change still applied to branches/publican-2x.
The fix will apply in 3.0 since we changed DocBook5 to be a tech preview.
Confirmed, <imageobject> and <graphic> render the same.
OS: Fedora 16 Publican Version: publican-3.0-0.fc16.t168.src.rpm Remarks: double-confirmed. <graphic fileref="image.svg" /> results in graphic displaying correctly.