This means node teasers do not get colorized like other text (e.g. the full node view).

I can't believe its meant to be like this so its either a regression or was never added correctly to begin with (not sure).

CommentFileSizeAuthor
#5 d7.932958.patch308 byteshaza
#2 teaser.png38.23 KBJeff Burnz
#2 full-node.png43.49 KBJeff Burnz

Comments

droplet’s picture

droplet commented

any example ??

Jeff Burnz’s picture

Jeff Burnz commented
StatusFileSize
new full-node.png43.49 KB
new teaser.png38.23 KB

Why we need examples I am not sure, but whatever.

Teaser

bartik teaser text is not recolor

Full node view

full node text is correctly recolored

droplet’s picture

droplet commented

hmm.. I can't reproduce it.

Jeff Burnz’s picture

Jeff Burnz commented

Its in the CSS, hard coded:

.node-teaser .content {
  color: #3b3b3b;
  clear: none;
  line-height: 1.6;
}

What you saying then - the CSS doesn't really exist and the I just made the issue up for a laugh? Do you understand how to recreate this?

This should not be where it is, it should be in colors.css or not exist at all (simply removing it solves the issue).

haza’s picture

haza
Primary language French
commented
Status: Active » Needs review
StatusFileSize
new d7.932958.patch308 bytes

Confirmed here.

As Jeff said, what about just removing the color in style.css ?

Jeff Burnz’s picture

Jeff Burnz commented
Priority: Normal » Major

Bump to major, can't ship like this, its a major flaw in the "colorability" of the theme.

dcrocks’s picture

dcrocks commented

Tested in FF 3.6, Safari 5, Opera 10.6, and chrome. Does what you would expect.

yoroy’s picture

yoroy commented
Status: Needs review » Reviewed & tested by the community

Good catch, quick fix: lots of win.

dcrocks’s picture

dcrocks commented

What is needed to get this committed, especially as it is 2 for 1.

Jeff Burnz’s picture

Jeff Burnz commented

#5: d7.932958.patch queued for re-testing.

jensimmons’s picture

jensimmons commented

I agree that this is a bug. I confirmed the bug, applied the patch cleanly, and successfully recolor the node teaser text along with the other body text and the block text — all as one color setting. I used Firefox 3.6.12 for Mac.

Looks good to me.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

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