This would mock the HTML 5 tag .

CommentFileSizeAuthor
#1 replace_navbar_with_nav2.patch20.69 KBn8tron
replace_navbar_with_nav.patch20.06 KBn8tron

Comments

n8tron’s picture

n8tron
he/him
Primary language English
Location Indiana
commented
StatusFileSize
new replace_navbar_with_nav2.patch20.69 KB

Rerolled. Pardon missed one.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Component: Miscellaneous » CSS/HTML Markup
Status: Needs review » Needs work

Hmm… we've got an inconsistency here between the HTML5 spec and Drupal standards. HTML5 uses "nav" and Drupal standards say we should never use abbreviations for variable names.

How about this? We use <div id="nav">, but all the variables and regions should be an un-abbreviated "navigation". Do you want to re-roll using the latest CVS? (I committed a bunch of stuff recently.)

jix_’s picture

jix_ commented
Status: Needs work » Fixed

Seems to be fixed already? It's div#navigation now …

Status: Fixed » Closed (fixed)

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