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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

droplet’s picture

droplet CreditAttribution: droplet commented

any example ??

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented
FileSize
full-node.png43.49 KB
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 CreditAttribution: droplet commented

hmm.. I can't reproduce it.

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: 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
CreditAttribution: Haza commented
Status: Active » Needs review
FileSize
d7.932958.patch308 bytes

Confirmed here.

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

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: 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 CreditAttribution: dcrocks commented

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

yoroy’s picture

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

Good catch, quick fix: lots of win.

dcrocks’s picture

dcrocks CreditAttribution: dcrocks commented

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

Jeff Burnz’s picture

Jeff Burnz CreditAttribution: Jeff Burnz commented

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

jensimmons’s picture

jensimmons CreditAttribution: 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 CreditAttribution: 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.