Great module - I'm already using it on two sites. However, I have run into a problem with the image code it produces being invalid. There is no alt tag, and no closing slash (necessary for XHTML).

A valid image tag should look like this:

<img src="image-source.jpg" alt="alt text" />

But the code this module provides looks like this:

<img src="image-source.jpg">

Any chance you could set this up to produce valid code? It would also be nice if the alt tag were customizable per image, though that's a secondary issue to the valud code.

Either way, for now I have changed the code on my own to produce valid code, but if you release updates, I would like to not have to change the code with each update. Thank you.

Comments

tamerzg’s picture

Category: bug » feature

Ok, I will change this into feature request. If you can share your code, maybe we can add it.

jaypan’s picture

Sure, give me a day or so, I'll put it up for you. Thanks!

getstarted’s picture

Hi Jay Natwichuk,
I need help, I am not getting the specific images for pages when i logout, when logged in i can see specific images for specific page.
Ex:for about page I can about.jpg as banner, but when i logged out I don't see about.jpg instead i am getting default.jpg for all pages.
Do i need to set anything?
I saw your post and you said it is working for you. please help me to fix this.
or give me instructions how to set up from start.
Thanks in advance
Padma