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.

CommentFileSizeAuthor
#7 div-submitted-after-patch.png17.15 KBsivaji_ganesh_jojodae
#7 div-submitted-before-patch.png17.14 KBsivaji_ganesh_jojodae
#7 658704_node_meta_distinct-7.patch477 bytessivaji_ganesh_jojodae
make_node_meta_distinct_from_body.patch493 bytessivaji_ganesh_jojodae
node_meta_before_patch.png33.88 KBsivaji_ganesh_jojodae
node_meta_after_patch.png33.56 KBsivaji_ganesh_jojodae

Comments

cosmicdreams’s picture

cosmicdreams commented

Why did you have to include:

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

sivaji_ganesh_jojodae commented

That has got nothing to do with my patch.

seutje’s picture

seutje commented
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

sivaji_ganesh_jojodae commented

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

seutje commented

yeh sry, by height I mean font-size

ff1’s picture

ff1 commented
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

sivaji_ganesh_jojodae commented
Status: Needs work » Needs review
StatusFileSize
new 658704_node_meta_distinct-7.patch477 bytes
new div-submitted-before-patch.png17.14 KB
new div-submitted-after-patch.png17.15 KB

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

tobiasb’s picture

tobiasb
Primary language German
Location Berlin
commented
Status: Needs review » Reviewed & tested by the community
dries’s picture

dries commented
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.

Issue tags: +Needs usability review

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.