WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 29792
16556
Page div layers do not render in correct location. They end up overlapped.
https://bugs.webkit.org/show_bug.cgi?id=16556
Summary
Page div layers do not render in correct location. They end up overlapped.
Adam Williams
Reported
2007-12-21 08:02:59 PST
comparison browser was Firefox 2.x which did not exhibit the same behavior. Safari Version 3.0.4 (5523.10.6) also shows bad layout behavior.
Attachments
Add attachment
proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1
2007-12-21 09:49:37 PST
Confirmed with a local debug build of WebKit
r28925
with Safari 3.0.4 (523.12) on Mac OS X 10.4.11 (8S165).
Ingo Chao
Comment 2
2007-12-21 14:26:01 PST
They do some heavy browser sniffing in wikibits.js and load KHTMLFixes.css, so #column-content { margin-left: 0; } in KHTMLFixes.css erroneously overrides #column-content { margin-left: 200px; } in main.css
David Kilzer (:ddkilzer)
Comment 3
2007-12-21 16:54:28 PST
(In reply to
comment #2
)
> They do some heavy browser sniffing in wikibits.js and load KHTMLFixes.css, so > #column-content { margin-left: 0; } in KHTMLFixes.css erroneously overrides > #column-content { margin-left: 200px; } in main.css
Thanks, Ingo! Based on this information, changing this to an evangelism bug.
David Kilzer (:ddkilzer)
Comment 4
2007-12-21 19:15:11 PST
Here's the faulty is_khtml detection code: var is_khtml = (navigator.vendor == 'KDE' || ( document.childNodes && !document.all && !navigator.taintEnabled )); Here's the code that writes the unwanted stylesheet: // document.write special stylesheet links if(typeof stylepath != 'undefined' && typeof skin != 'undefined') { if (is_opera_preseven) { document.write('<link rel="stylesheet" type="text/css" href="'+stylepath+'/'+skin+'/Opera6Fixes.css">'); } else if (is_opera_seven) { document.write('<link rel="stylesheet" type="text/css" href="'+stylepath+'/'+skin+'/Opera7Fixes.css">'); } else if (is_khtml) { document.write('<link rel="stylesheet" type="text/css" href="'+stylepath+'/'+skin+'/KHTMLFixes.css">'); } }
David Kilzer (:ddkilzer)
Comment 5
2007-12-21 19:24:30 PST
Easy fix for wikibits.js: - } else if (is_khtml) { + } else if (is_khtml && !is_safari) {
Dimitri Glazkov (Google)
Comment 6
2009-10-31 10:10:23 PDT
Fixed in
bug 29792
. *** This bug has been marked as a duplicate of
bug 29792
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug