Bug #31
Root styles
| Status: | Closed | Start: | ||
|---|---|---|---|---|
| Priority: | High | Due date: | ||
| Assigned to: |  Jeremy Bush |
% Done: | 0% |
|
| Category: | Argentum | |||
| Target version: | Alpha Release | |||
| Resolution: | wontfix |
Description
Things like this:
<div id="header" class="clear">
should be avoided.
I don't like the idea of having to remove and reapply a class all over the place because it was locked in so tightly in the first place.
If you're going to assign something an ID, apply all the styles in the #id definition, don't duplicate that class all over the place.
I don't know how to describe it other than an incidental style. This is not a type of content or anything. It's like having something have a bold-text class. Yes, it creates extra code in the style sheet, but removes a nightmare later on for people that don't want to deal with that. If you want to clear: on the header, and the logo, then do it in the #id definition. Don't give them all a class incidental to the individual needs, when they don't match a specific type of content.
History
Updated by Kevin Chan - over 3 years ago
- Status changed from New to Closed
- Resolution set to wontfix
This is a trick to clear floats without cluttering the style sheet. This is commonly used throughout the application. Being DRY is more important than being schematic in this case.
It is no way a "nightmare".