Further formatting separation in Kubrick theme

[brettz95]

I was told to report this issue here (the theme is for one of the defaults).

Although I understand HTML5 is considering keeping the <small/> tag for the sake of things like legalese fine print, when text is made small merely due to a stylistic choice, it is considered best practice to separate the styling in the stylesheet. This lets structural code stay correct even if we wish to make our text no longer small.

For the Kubrick theme, in style.css, I'd suggest adding the following (since span won't inherit the current color otherwise):

.permlink {

color:#777777;

}

and then add .permlink to the small declaration (or use it in place of 'small' if you're not using 'small' elsewhere):

small, .permlink {

font-family:Arial,Helvetica,Sans-Serif;
font-size:0.9em;
line-height:1.5em;

}

Then in index.php (in place of where <small/> was used):
<span class="permlink"><?php the_time('F jS, Y') ?> <!-- by <?php the_author() ?> --></span>

[Denis-de-Bernardy]

sending this to the will think about it in a few years bin.

[brettz95]

For the amount of time you spent changing the priorities, you could have implemented what should be a very non-controversial patch. While it admittedly is not that big of a deal, I would think WordPress, in priding itself on good practices, attention to detail, and standards compliance, wouldn't have a problem to make a very easy fix for better stylesheet flexibility.

[Denis-de-Bernardy]

Well, that's the motto. For my part, I punted this in under 20s from lack of patch, and I couldn't care less if my site doesn't validate.

[brettz95]

Moving small formatting to style sheet

[brettz95]

Ok, I've added a patch, and it covers all pages in the theme which use "small". But this particular patch wasn't about validation--just separation of structure from formatting. FWIW, the original creator of the theme stated he "heartily agreed" with my changes to keep them separate as I proposed. Thanks much...

[johnbillion]

n/a now we have Twenty Ten

[iandstewart]

Going to mark this as wontfix out of concern for those using Default as a Child theme and expecting the markup to stay the same.

[brettz95]

First, let me apologize to Denis for being a jerk in my earlier comment. I think I had just spent some time on the patch and was assuming something like that would make it in, but not accounting for the fact that everybody is busy, etc. I guess it's just that for contributors who care about the code, a "wontfix" or equivalent is not a fun thing to run into.

That being said, are that many authors really going to be depending on modifying the default styles for these smaller issues? It's not like these are modifying whole divs or anything, and my style changes were meant to replicate the same styling as before, just separating the formatting into the stylesheets where they belong. The author of this theme was himself in favor of the changes. And like other upgrades that sometimes need to break things a little bit if you can even call it that, this will give more control/clarity to users (and the type of users who have modified their stylesheets probably understanding the reasoning for the changes anyways and can easily fix whatever they've done).

I'll leave it at that, but I think it would be nice to go 100%. Thanks...