The node meta ("Submitted by admin on Wed, 11/25/2009 - 17:29") looks similar to body. For instance see the attached snapshot. IMO they need to be distinct as in garland theme. Attached snapshot will do the needful.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cosmicdreams’s picture

Why did you have to include:

ul.secondary li.active a.active {
   min-height: inherit;
 }
Sivaji_Ganesh_Jojodae’s picture

That has got nothing to do with my patch.

seutje’s picture

Issue tags: +Needs usability review

the height change seems rather insignificant, do you really think it's necessary?

tagging for UX

Sivaji_Ganesh_Jojodae’s picture

My patch is to add the following lines to style.css, nothing more.

+.meta span.submitted {
+  color: #898989;
+  font-size: 0.92em;
+}

To be frank it just duplicates the stuff that already exists in garland theme.

seutje’s picture

yeh sry, by height I mean font-size

ff1’s picture

Status: Needs review » Needs work

The .meta class does not exist anymore.
The .submitted class is now within a div.

So the patch should be:

div.submitted {
  color: #898989;
  font-size: 0.92em;
}
Sivaji_Ganesh_Jojodae’s picture

Status: Needs work » Needs review
FileSize
477 bytes
17.14 KB
17.15 KB

@ff1 Thanks for correcting, attached patch will do the needful.

tobiasb’s picture

Status: Needs review » Reviewed & tested by the community
Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Needs usability review

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