In system.css there is the following bit of css

/* Avoid jumping around due to margins collapsing into collapsible fieldset border */
html.js fieldset.collapsible .fieldset-wrapper {
  overflow: auto;
}

This is causing weird side effects (shown in attached screenshot).

Despite the comment, I do not see any differences with or without this css, and because it is causing additional issues, I see no reason why it should not be removed. I would appreciate additional testing to make sure I am not missing any side-effects of removing this bit of code.

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

ultimateboy’s picture

Status: Needs work » Needs review

Lets try this again...

Status: Needs review » Needs work

The last submitted patch failed testing.

ultimateboy’s picture

Status: Needs work » Needs review
StatusFileSize
new597 bytes

Re-roll.

ultimateboy’s picture

StatusFileSize
new597 bytes

Re-roll

cburschka’s picture

Needs moar testing, I believe.

Although the directive deserves to be axed just for having such an obscure comment. "Jumping around"? How? When? And in what browser? :P

Speaking of which, what browser is that screenshot from? I've never ever seen it look like this.

catch’s picture

Status: Needs review » Reviewed & tested by the community

This was added in 2006 to fix a Safari issue. That was probably Safari 1 or 2, so let's just remove it.

catch’s picture

Status: Reviewed & tested by the community » Needs work

The last submitted patch failed testing.

casey’s picture

Status: Needs work » Needs review
StatusFileSize
new636 bytes
casey’s picture

Status: Needs review » Closed (duplicate)

On second thoughts, setting this a dupe is more efficient.