Terminate CSS comment in layout.css

Looks like cruft to me
Should have been commented if it were needed.
Roll away...

I'm not seeing this. Can you give me a line number or a context?

Hmm...

A quick look at layout.css shows this exists on or near line 96 (the very bottom) its a quick fix, i've attached a patch for review.

rport

Has this been committed yet?

This patch, which effectively uncomments the bottom 3 lines of layout.css, breaks the footer in the zen-fixed theme. It causes the footer to take up as much space as in the zen non-fixed theme.

I suspect from reading this that the area affected should have been commented out however the terminating */ was left out (see below).

Here is a copy of the relevant section of code;

/*
body.sidebar-left  #footer {
  margin-left: -210px;
}

body.sidebar-right #footer {
  margin-right: -210px;
}

body.both-sidebars #footer {
  margin: 0 -210px;
}

Over to jjeff to consider this and resolve the matter...

1. Is this some legacy code that should be removed, not just commented out; or
2. Is this css for some new feature/idea that is simply missing the block terminator; or

~ rport

Subscribing

Here is a proper patch. Works in both fixed and full modes.

committed to 5 dev