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.

Files: 
CommentFileSizeAuthor
#7 div-submitted-after-patch.png17.15 KBSivaji
#7 div-submitted-before-patch.png17.14 KBSivaji
#7 658704_node_meta_distinct-7.patch477 bytesSivaji
PASSED: [[SimpleTest]]: [MySQL] 24,825 pass(es). View
make_node_meta_distinct_from_body.patch493 bytesSivaji
Passed on all environments. View
node_meta_before_patch.png33.88 KBSivaji
node_meta_after_patch.png33.56 KBSivaji

Comments

cosmicdreams’s picture

Why did you have to include:

ul.secondary li.active a.active {
   min-height: inherit;
 }
Sivaji’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’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’s picture

Status: Needs work » Needs review
FileSize
477 bytes
PASSED: [[SimpleTest]]: [MySQL] 24,825 pass(es). View
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.