Biblio nodes not XHTML-compliant

Dane Powell - January 9, 2009 - 02:12
Project:Bibliography Module
Version:6.x-1.0-rc2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Individual biblio nodes do not validate as XHTML 1.0 Strict due to valign, width, and align tags used in the title cell of each table row. I created a class biblio-td-left in biblio.css and moved the properties into this class. I have included a patch against 6.x-1.0-rc2.

I don't know if this is the proper solution or not - this is my first time submitting a patch for a module. Let me know if you have any suggestions or better ideas.

AttachmentSize
biblio_xhtml_patch.patch1.12 KB

#1

Dane Powell - January 12, 2009 - 17:45
Status:active» needs review

I'm going to take the liberty of updating this to "needs review," since no one has made any alternative suggestions...

#2

rjerome - January 12, 2009 - 18:23

I would have sworn that I already replied to this, but it seems to have been lost in the ether.

So again, thanks for the patch and I didn't see any problem with it and committed it to CVS last on the weekend.

Ron.

#3

Dane Powell - April 24, 2009 - 00:09
Status:needs review» fixed

#4

System Message - May 8, 2009 - 00:10
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.