There's quite a lot of stuff to clean up in theme.css since 6.x-2.x-beta2. Rather than issues and patches for each little thing, I'll just list them in comments here, as I find them...

Comments

andrewmacpherson’s picture

andrewmacpherson commented
Version: 6.x-2.x-dev » 6.x-2.0-beta2
Status: Active » Needs review

I'll refer to theme.css line numbers from from beta2

andrewmacpherson’s picture

andrewmacpherson commented

line 154 refers to sidebar-inner, which has gone:
/* Target a specific sidebar, inner or sidebar-inner. */

change to:
/* Target a specific sidebar. */

andrewmacpherson’s picture

andrewmacpherson commented

line 195 says "remove display:block..."

Remove implies there's something to delete; a better wording would be:
/* Over-ride display:block if you want the site name inline with the logo. */

andrewmacpherson’s picture

andrewmacpherson commented

line 451: What is .tags {} used for? I can't find the class in the HTML. The name suggests taxonomy, but it doesn't appear in the node taxonomy output.

andrewmacpherson’s picture

andrewmacpherson commented

lines 479-482 and 484-487:

.article ul.links selectors are repeated, only need one set.

andrewmacpherson’s picture

andrewmacpherson commented

line 511-513:

.first and .last classes for ul.links ?

These don't seem to be produced any more. Could we have them back, please? Perhaps another theme setting...

andrewmacpherson’s picture

andrewmacpherson commented

line 496 refers to
.article h3.vocab-name {}

However it would be a h2 element on standalone node/* pages. Perhaps it should be generalized to:
.article .vocab-name {}

starter.css (line 55) just uses this form.

andrewmacpherson’s picture

andrewmacpherson commented

line 497-499:

replace ul.terms with ul.vocab-list

Jeff Burnz’s picture

Jeff Burnz commented

Cheers, I've nailed everything so far and committed it so its in DEV.

I cant remember when I last used .tag, gawd, I think that was way back when...

Adding back in first and last just for node & comment links, hmm, not even sure how to do that, but if you enable .first and .last for Menu items in the theme settings these apply to the links lists.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Needs review » Fixed

I think we can call this fixed for now, if more are found we can reopen it.

Status: Fixed » Closed (fixed)

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