Closed (fixed)
Project:
footermap: a footer site map
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Dec 2010 at 12:44 UTC
Updated:
21 Jan 2011 at 19:00 UTC
change footermap output to render valid xhtml strict? currently output is multiple IDs for footermap div...?
here is the error i got- also see attached...
Line 284, Column 74: value of attribute "id" must be a single token
in footermap.module
so my quick fix was just to remove the space in between the current output of the id based on $n-
and the one based on $key-
and add a dash so the two separate IDs would instead be rendered as one, hyphenated merged ID.
seems to work fine...
before:
$o .= '<div class="footermap-col" id="footermap-col-'.$n.' footermap-col-'.$key.'">'.theme('links', array('links' => $block, 'heading' => array('text' => t($heading), 'level' => 'h3'), 'attributes' => array('class' => 'footermap-item links'))) . '</div>';
after:
$o .= '<div class="footermap-col" id="footermap-col-'.$n.'-footermap-col-'.$key.'">'.theme('links', array('links' => $block, 'heading' => array('text' => t($heading), 'level' => 'h3'), 'attributes' => array('class' => 'footermap-item links'))) . '</div>';
| Comment | File | Size | Author |
|---|---|---|---|
| footermap_validation.png | 36.37 KB | basicmagic.net |
Comments
Comment #1
mradcliffeYes, looks like a typo. Thank you. I committed it.