Changes between Initial Version and Version 1 of Ticket #33755, comment 134
- Timestamp:
- 03/05/2016 12:50:45 AM (9 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #33755, comment 134
initial v1 8 8 I 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. 9 9 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.10 Another 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.