The <!-- /#name-and-slogan --> style HTML comments used to be useful when our primary debugging tool was "view source". But with the rise of web-based inspectors, they are just code bloat.

Let's remove them from all our template files.

CommentFileSizeAuthor
#2 zen-1716718-2.patch4.62 KBstar-szr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

star-szr’s picture

Assigned: Unassigned » star-szr

I'll work on a patch for this.

star-szr’s picture

Status: Active » Needs review
FileSize
4.62 KB

The patch leaves only IE conditional comments, removing these closing comments as well:

<!-- block__no_wrapper -->
<!-- region__footer -->
<!-- region__no_wrapper -->
<!-- region__sidebar -->

From block--no-wrapper.tpl.php, region--footer.tpl.php, region--no-wrapper.tpl.php, region--sidebar.tpl.php.

JohnAlbin’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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