Columns layout and visibility
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
UD Theme |
Fix Released
|
Low
|
Michael Lustfield |
Bug Description
Follows bug #379084
Now that most (sigh) css rules are in css file, I'll try to help fixing them.
== Unused or misnamed container ids ==
Looking at page.tpl.php, I see that the main content is organised with
the following hierarchy:
#container
$breadcrumb
#container3
so selectors with #container0, container-1, container-2, container-3 will be ignored.
I removed them in the patch below.
Also, #container3 and #container2 are just bare wrappers (no extra content) which
just add confusion in the stylesheet. I propose to remove all rules for them,
as well as the elements themselves.
== Misuse of "right" css property ==
See http://
"This property specifies how far a box's right content edge is offset to the left of the right edge of the box's containing block. "
Then in #container1, I read
width: 100%;
right: 75%;
This implies that left is now -75%, and it sadly means that the offsets of all inner elements have to balance this strange offset.
Recall that width percentage is calculated with respect to the width of the generated box's containing block. ( http://
== overflow settings ==
Most rules in the layout section define "overflow=hidden".
This makes the errors related to above one, very critical !
http://
Moreover, this property's (as well as "position"'s) default value is "inherit", so there's no need to repeat it in inner elements.
http://
Also, I'm not sure that defining overflow where width=100% makes sense.
== Columns positionning ==
Why not leaving "float:left" doing the job of setting the "left" value ? just set "width"
I'd tend to see float and left as conflicting ways of specifing horizontal position of the box.
Better choose one and stick to it.
As I prefer the more modern fluid layouts, I'd try float.
== The patch ==
Apply patch for bug #379084 first !
Values are only rough settings, there is some work to get a good rendering, I leave this to you.
But I'm sure it should be far easier with this new structure.
Related branches
- Michael Lustfield (community): Disapprove
- Diff: None lines
Changed in ubuntu-drupal-theme: | |
assignee: | nobody → Rocky Road (m-baert) |
importance: | Undecided → Low |
status: | New → Triaged |
description: | updated |
description: | updated |
Changed in ubuntu-drupal-theme: | |
status: | Triaged → Fix Committed |
on #container add: "padding-left: 17;