I am testing the D7 version of zen and I just noticed the nodes.css file includes a ".node .picture" which should style the picture of the node author (as written in the comment of the class) but actually the "div" that englobes the user picture has the class ".user-picture" instead.

Shouldn't that be changed to ".node .user-picture"?

Thanks!
David.

CommentFileSizeAuthor
#2 zen-HEAD-1298094941.patch1.37 KBhswong3i
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

davebv’s picture

davebv CreditAttribution: davebv commented
Title: .picture class for user pictures » .picture class for user pictures should be .user-picture

(Change the issue title to be more descriptive)

hswong3i’s picture

hswong3i CreditAttribution: hswong3i commented
Priority: Normal » Major
Status: Active » Needs review
FileSize
zen-HEAD-1298094941.patch1.37 KB

+1 for following changes:

  • ".node .picture " => ".node .user-picture"
  • ".comment .picture => ".comment .user-picture"

Patch via CVS HEAD, tested with http://drupal.org/project/interactive_media Drupal 7.x Zen 3.x development.

JohnAlbin’s picture

JohnAlbin
he/they/他
Primary language English
Location Taipei, Taiwan
CreditAttribution: JohnAlbin commented
Title: .picture class for user pictures should be .user-picture » User pictures should use .user-picture class
Status: Needs review » Fixed

Thanks for the patch! :-)

Fixed. http://drupal.org/cvs?commit=502436

Status: Fixed » Closed (fixed)

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