WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
121419
[GTK] [Stable] [Regression] document.body.scrollTop is always 0
https://bugs.webkit.org/show_bug.cgi?id=121419
Summary
[GTK] [Stable] [Regression] document.body.scrollTop is always 0
Guillaume Desmottes
Reported
2013-09-16 03:21:04 PDT
Original bug report:
https://bugzilla.gnome.org/show_bug.cgi?id=707781
- Start an empathy chat and reduce the window size so the vertical scrollbar appears - Move up the scrollbar so the latest lines aren't visible - Send a message Expected result: the window scroll down so the sent message is visible Actual result: the window doesn't scroll down Empathy uses a nearBottom() JS function (empathy/data/Template.html) to check if it should automatically scroll down when a new message is displayed. This function is broken because document.body.scrollTop is always 0, ignoring the actual position of the scrollbar. This can also be easily reproduced using the WebKit inspector to check the value of scrollTop. Just use 'gsettings set org.gnome.Empathy.conversation enable-webkit-developer-tools true' and then right click in the chat window. I can only reproduce this bug using webkit-2.2 branch. It works fine using 2.0.4 so that's probably a regression.
Attachments
Add attachment
proposed patch, testcase, etc.
Zan Dobersek
Comment 1
2013-09-16 07:39:09 PDT
document.body.scrollTop and document.body.scrollLeft are returning 0 if the document is not in quirks mode after
r154614
.
http://trac.webkit.org/changeset/154614
The simplest way to enter the quirks mode is to remove the DOCTYPE or change it into something that would induce the quirks mode, for instance <!DOCTYPE html PUBLIC> Could you give that a go?
Guillaume Desmottes
Comment 2
2013-09-16 08:27:29 PDT
Indeed removing the DOCTYPE solves this problem. I guess that's not really a bug then?
Zan Dobersek
Comment 3
2013-09-16 08:38:48 PDT
No, this is an intended change in behavior.
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