Make WordPress Core

Changes between Initial Version and Version 1 of Ticket #33755, comment 134


Ignore:
Timestamp:
03/05/2016 12:50:45 AM (9 years ago)
Author:
fatmedia
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #33755, comment 134

    initial v1  
    88I agree it can be useful, but I'm not sure it's important for core to add it by default. If a theme does need it for one reason or another, it's pretty quick to filter it in there.
    99
    10 IMO it's also probably something that would be more useful when applied to the parent container around the logo rather than the body class. The rest of the site likely won't care whether a logo is present or not, but the site's main `<header>` element might, just an example.
     10Another reason it probably makes sense to leave it out of core is that IMO a conditional class would be more useful when applied to the parent container around the logo. The rest of the site likely won't care whether a logo is present, but the site's main `<header>` element might, just an example.