After #1369676: Proposed theme changes to Origins lands, let's implement the following tweaks suggested by noyz:
- Seems like extra spacing between group avatars: https://skitch.com/ezra-g/g416r/our-community-commons-2.4-dev
- Ideally turn tabs into pills, eg https://skitch.com/jeff.noyes/g412m/jeff-noyes-jeff-noyes
- Make "submitted by" text less prominent in the entire theme, per https://docs.google.com/a/acquia.com/document/d/1IqS5vuKsSv0cDwk0vZGxgBM... . Ideally, this would happen with a tooltip per https://docs.google.com/a/acquia.com/leaf?id=0B6CiMWdwdDYJMmFmMmE3ZjYtMj.... Marking the issue as "needs more info" so that noyz can comment.
Comment | File | Size | Author |
---|---|---|---|
#3 | origins_tweaks-followup-1412084-03.patch | 1.53 KB | ay13 |
Comments
Comment #1
ezra-g CreditAttribution: ezra-g commentedComment #2
ezra-g CreditAttribution: ezra-g commentedLet's move " Make "submitted by" text less prominent in the entire theme, per" to a separate issue. Setting this as active for the remaining changes.
Comment #3
ay13 CreditAttribution: ay13 commentedattached patch fixes the first issue and adds the pill tabs for the second issue. i will need some guidance as to the coloring of the tab/pills though. the mock didn't look like it was from the origins theme so didn't know what the styling should be.
Comment #4
lightsurge CreditAttribution: lightsurge commentedI don't really understand the extra spacing between the avatars is an improvement, just adds blank space... I think mainly their weakness is that if a user has no avatar set, they look dull and non-descript.
Comment #5
ay13 CreditAttribution: ay13 commentedthe problem before was that the cells where not given a width. what would happen is that if there was just 2 members of a group those 2 would take over the width of an area messing up the spacing. by adding a width to the cells of the grid (percentages, 20% for 5 column etc.) every avatar has its own space.
Comment #6
ezra-g CreditAttribution: ezra-g commentedLooks good - thanks!
http://drupalcode.org/project/commons.git/commit/13b967b
Comment #7
ezra-g CreditAttribution: ezra-g commentedI accidentally overwrote the commit in #6 with the main commit for the new group homepage: http://drupalcode.org/project/commons.git/commitdiff/4f63c54a94849c61ba5...
Sorry about that.
I committed this patch again:
http://drupalcode.org/project/commons.git/commit/90169d6