This patch removes the extraneous markup when outputting menus in Drupal. It removes the uneeded wrapper <div>s, as all of the styles can be applied the <ul> directly. After this patch, menus will look *exactly* the same in core, however, there will be less markup, meaning faster loading pages, less tag-soup, and easier time for themers.

This is patch #2 in a series of patches to clean up the HTML spaghetti in Drupal.

Patch #1: http://drupal.org/node/15609#comment-63995

CommentFileSizeAuthor
drupal_32.patch2.48 KBm3avrck
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

chx CreditAttribution: chx commented

yummy! less code in Drupal AND in the HTML source code too. Great work.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Mad Maks’s picture

Mad Maks CreditAttribution: Mad Maks commented

This patch causing problems in the theme spreadfirefox, patch will be committed when this theme is ready for 4.7

Anonymous’s picture

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