There's a space tacked on to the class, so it comes out like this: class=" flickr-photo-img"

It's a little blip, but it gets in the way of attempts to apply styles.

Thanks.

CommentFileSizeAuthor
#5 flickr_147152.patch870 bytesdrewish

Comments

drewish’s picture

it really shouldn't affect the class name. you can apply multiple classes in a single class="foo bar" attribute... or am i wrong?

andrewlevine’s picture

you can apply multiple classes like that, but i wouldn't be suprised if some browsers choked on a space at the start.

drewish’s picture

did this get fixed?

andrewlevine’s picture

Assigned: Unassigned » andrewlevine

drewish, no, i don't think this got fixed. i wasn't sure if this was actually a problem so i left it.

we're probably better off just fixing it either way, do you agree? i'll do it in the next couple days if you do.

drewish’s picture

Status: Active » Needs review
StatusFileSize
new870 bytes

i haven't had a chance to test it out but i'm wondering if something like this would do the trick.

andrewlevine’s picture

Status: Needs review » Fixed

went ahead and committed this, r1.25.2.5

Anonymous’s picture

Status: Fixed » Closed (fixed)