[buddypress-trac] [BuddyPress] #2748: Add .bypostauthor class CSS

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Sat Nov 20 09:34:24 UTC 2010


#2748: Add .bypostauthor class CSS
-------------------------+--------------------------------------------------
 Reporter:  karmatosed   |       Owner:     
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  1.3
Component:  Theme        |     Version:     
 Keywords:               |  
-------------------------+--------------------------------------------------

Comment(by karmatosed):

 Cool that works and like it a lot.

 My only concern is the notes from Theme Review are not clear and say
 include class but don't say if has to be a full class.  That said I don't
 see the argument against a reference like this as it's still showing what
 is a post author.

 I agree though in hindsight not sure a full on class works with the
 current comment styling at all.  Unless you add padding and have same
 background to border (then loosing border) like I did example it looks
 odd.  I guess only running through test will tell us if this is acceptable
 - I as said think worth questioning why if fails.

 Another option thinking this morning would be changing border colour on
 the avatar.

-- 
Ticket URL: <http://trac.buddypress.org/ticket/2748#comment:4>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list