Still tweaking things for the site. It would be great if there were CSS controls for the different elements of an image post. I see CSS classes for the image and the thumbnail. It would be cool if the body and teaser also had classes.

Thanks Again!
- Joe Lombardo

Comments

TheLibrarian’s picture

TheLibrarian commented

On this note, the image.module should remove the "border", "align", "hspace", and "vspace" attributes from Only local images are allowed. tags and place them in css. Currently, these tags do not qualify as valid HTML 4.01 (or XTHML 1.0).

bruno’s picture

bruno commented

Check with current cvs.

Robrecht Jacques’s picture

Robrecht Jacques commented
Component: Code » image.module
Status: Active » Closed (fixed)

I believe this is fixed in current CVS-HEAD: "img" tags get the necessary classes (eg: class="image preview" or class="image thumbnail"). As the rendering code for galleries has changed not to use tables, things like "valign" etc don't appear anymore.

If you think the CSS/HTML is still not "right", please reopen this issue after you've check the latest CVS-HEAD version.