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.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | fieldsetoverflow.patch | 636 bytes | casey |
| #5 | 367367_remove_overflow_css-4.patch | 597 bytes | ultimateboy |
| #4 | 367367_remove_overflow_css-4.patch | 597 bytes | ultimateboy |
| css_overflow_issue.png | 11.19 KB | ultimateboy | |
| remove_overflow_css.patch | 600 bytes | ultimateboy |
Comments
Comment #2
ultimateboy commentedLets try this again...
Comment #4
ultimateboy commentedRe-roll.
Comment #5
ultimateboy commentedRe-roll
Comment #6
cburschkaNeeds 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.
Comment #7
catchThis was added in 2006 to fix a Safari issue. That was probably Safari 1 or 2, so let's just remove it.
Comment #8
catchhttp://drupal.org/node/92849#comment-464156 for reference.
Comment #10
casey commentedReroll
#745380: CSS contains too many needless differences between JS and no-JS versions however also contains this.
Comment #11
casey commentedOn second thoughts, setting this a dupe is more efficient.