column width oddness

joachim - June 19, 2009 - 13:42
Project:Clean
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

I'm confused about this. Having a 700 and a 710 just doesn't look healthy to me:

.two-sidebars #centre {
  width: 500px;
}

.sidebar-left #centre {
  width: 710px;
  padding-right: 0;
}

.sidebar-right #centre {
  width: 700px;
}

#1

joachim - June 20, 2009 - 12:11

Yeah, you've basically got 20px instead of 10 between the centre and the right sidebar region.

Is it worth fixing this, or should we switch to use the 960 framework?
Even if we stay with the current approach, I'd be in favour of changing the width to 960px anyway as it divides more nicely than 910.

#2

joachim - June 20, 2009 - 12:12
Category:task» bug report

#3

psynaptic - June 21, 2009 - 15:16

We should fix the bug in Clean and add 960 as an extra. I don't usually work to grids any more since I always get a design send over by a designer who doesn't use grids.

I've used them before and if I was doing the design I would use a grid so I'd be more than happy to add it as an extra but not to completely take over as that is provided by other modules already.

#4

joachim - June 23, 2009 - 20:04
Status:active» fixed

Committed the following patch after testing with all column combinations.

AttachmentSize
clean_column_padding_496266.patch 692 bytes

#5

psynaptic - June 23, 2009 - 21:50

Awesome, thanks!

#6

System Message - July 7, 2009 - 22:00
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.