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>';
CommentFileSizeAuthor
footermap_validation.png36.37 KBbasicmagic.net

Comments

mradcliffe’s picture

Status: Active » Fixed

Yes, looks like a typo. Thank you. I committed it.

Status: Fixed » Closed (fixed)

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