I know this may seem minor, but the 1.4 version didn't have this problem. It looks like your nested divs inside "a href" tags is not allowed in XHTML. You could use spans I believe and get the same result if you change the css a little, or just add the ID tags to the "a href"'s themselves. Anyway, I've worked hard to keep a fully XHTML compliant website and your module breaks it. See attached screenshot. I know this can be fixed without disrupting too much. Please take a look.

See the site for more detail:
http://kpbj.com

See this official W3C validator output for more info:
http://validator.w3.org/check?uri=http%3a%2f%2fkpbj.com

CommentFileSizeAuthor
#2 rpx_widgets.module_20111110.patch398 bytesgregarios
janrain_errors.png280.1 KBgregarios

Comments

gregarios’s picture

gregarios commented
Priority: Minor » Normal
gregarios’s picture

gregarios commented
Title: Version 6.x-2.1 output not XHTML compliant » 6.x-2.1 (and 6.x-2.x-dev) not XHTML compliant (a patch to fix supplied)
Version: 6.x-2.1 » 6.x-2.x-dev
StatusFileSize
new rpx_widgets.module_20111110.patch398 bytes

Attached is a simple patch to fix this, if you wouldn't mind applying it to your 6.x-2.x-dev version. It fixes XHTML compliance with the widgets login module.

gregarios’s picture

gregarios commented

Any chance this will get into the next release??

geokat’s picture

geokat commented

I don't see why not.

gregarios’s picture

gregarios commented
Title: 6.x-2.1 (and 6.x-2.x-dev) not XHTML compliant (a patch to fix supplied) » 6.x-2.2 (and 6.x-2.x-dev) not XHTML compliant (a patch to fix supplied)
Priority: Normal » Major

For some reason this patch didn't get into the 6.x-2.2 version!

Having div tags inside of a tags is not W3C compliant.

geokat’s picture

geokat commented

6.x-2.2 was a security release.
This patch should be included in the next regular release.