Currently the module provides the <link> elements in either the footer or a block. That is nice, but unless the block is placed in the head it's not XHTML compliant.

This isn't great.

CommentFileSizeAuthor
#1 move_default_inclusion_from_footer_to_head.patch688 bytesguardian

Comments

guardian’s picture

guardian commented
StatusFileSize
new move_default_inclusion_from_footer_to_head.patch688 bytes

i moved the default inclusion from footer to the html header, see patch attached
cheers

oh and btw this patch doesn't include the fix for the non prefixed tables in queries

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Postponed » Needs review

This seems like a...creative... use of hook_help. Shouldn't this go into _init or _menu?

Thanks for the patch!

guardian’s picture

guardian commented

it's not using hook_help

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: make xhtml4.01 compliant » make xhtml4.01 compliant by moving microsummaries to head
Version: master » 5.x-1.0
Status: Needs review » Fixed

right you are - seems I misread it. I've committed this to the 5.x branch. When this gets ported it probably makes sense to copy over the 5 branch.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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