Project

General

Profile

Actions
Copy link

Bug #2038

closed

Fixing homepage spacing

Added by Matt Gold over 11 years ago. Updated over 11 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Matt Gold
Category name:
WordPress (misc)
Target version:
1.4.2
Start date:
2012-08-11
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

Hi All,

I know that this is going to be fixed in November/December when PressCrew creates our child theme of the CBox installation, but this is important enough that I think we should address it now if it wouldn't be too much work. In the attached screenshot, I've highlighted something that has bothered me for a while -- the overly large empty space between three elements (slide, featured content, who's online) on our homepage.

Is there anything we could do as a relatively quick fix to improve this situation? I know we're pressed for time on CBox, but if there is anything we can do on this, I'd really like to do it because our homepage is vitally important and some small changes could have a very large impact.

Files

Download all files
commons-homepage.jpg (357 KB) commons-homepage.jpg Matt Gold, 2012-08-11 11:15 AM
homepage_redeux.png (719 KB) homepage_redeux.png Chris Stein, 2012-08-13 01:21 PM
Actions
Copy link
 #2

Updated by Boone Gorges over 11 years ago

  • Assignee changed from Boone Gorges to Matt Gold

Assigning to Matt until we've arrived at some concrete changes - feel free to assign back to me when there's something to implement.

Actions
Copy link
 #3

Updated by Boone Gorges over 11 years ago

  • Target version changed from 1.4.1 to 1.4.2
Actions
Copy link
 #4

Updated by Chris Stein over 11 years ago

I took a quick look at it and here are some suggestions.
  • Take a Tour button, both padding set to 7px (one is 12px)
  • ake a Tour button,change the font-size from 16 to 13
  • Featured, use a little less characters so only 4 lines.
  • Featured, What Matt said was padding is actually where the "Read More..." text should be. But it isn't showing up on mine either (text is there in HTML). Instead why don't we just get rid of that text and make [...] into a link.
  • div.introbox should probably be 340px instead of 360px high to reflect actual height of the space
  • Right now cac_featured_content_widget-2 has a bottom-margin of 15px through styling on the .featured class (1744). Because the "Who's Online" text will always leave some gap between featured content and what's below then it can have it's margin changed to 0.
    #cac_featured_content_widget{
    margin-bottom:0;
    }

Those changes left it as seen in attached screenshot

Actions
Copy link
 #5

Updated by Matt Gold over 11 years ago

Wow -- that's fantastic, Chris. A million times better. It will be such a relief to have this fixed. Boone, please let us know what you think. I'm delighted.

Actions
Copy link
 #6

Updated by Boone Gorges over 11 years ago

Matt, is this a go from your point of view? Want to make sure we're set before implementing.

Chris, are there any additional CSS rules you changed, that you haven't listed above? I can reverse engineer, but would prefer not to :)

Actions
Copy link
 #7

Updated by Matt Gold over 11 years ago

Yes -- huge go, with the proviso that I haven't seen a working demo. I'd test it on CDev and implement straight away if it looks good.

Actions
Copy link
 #8

Updated by Chris Stein over 11 years ago

Boone, that should be it. If you run into an issue implementing it feel free to just put it up on CDev and ask me to take a look.

Actions
Copy link
 #9

Updated by Boone Gorges over 11 years ago

  • Status changed from Assigned to Resolved
Actions
Copy link

Also available in: Atom PDF