After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 707781 - new messages don't cause scroll
new messages don't cause scroll
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Chat
unspecified
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
empathy-maint
Depends on:
Blocks:
 
 
Reported: 2013-09-09 15:53 UTC by Martin Meyer
Modified: 2013-09-16 15:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Template.html: remove DOCTYPE (934 bytes, patch)
2013-09-16 15:19 UTC, Guillaume Desmottes
none Details | Review

Description Martin Meyer 2013-09-09 15:53:30 UTC
I'm using Empathy 3.9.91 with the "Boxes" theme currently in a fresh F20 pre-alpha install. Empathy seems to not be scrolling on receiving new messages right now. New messages are below the visible text area until I explicitly scroll down.

I'm pretty sure my install is "clean" (didn't keep dconf settings), but I did keep old logs and some adium themes (which I'm not using).
Comment 1 Guillaume Desmottes 2013-09-10 07:54:17 UTC
Does it happen with outgoing messages as well?

I've been told about an issue with recent versions of WebKitGtk. Which version are you using?
Comment 2 Martin Meyer 2013-09-10 13:42:37 UTC
Looks like it does happen with outgoing as well. My Empathy version is 3.9.91, my webkit package claims to be webkitgtk3-2.1.90.1.
Comment 3 Guillaume Desmottes 2013-09-13 14:12:58 UTC
I managed to reproduce using https://trac.webkit.org/wiki/WebKitGTK/2.2.x
I'll investigate the bug on Monday.
Comment 4 Guillaume Desmottes 2013-09-16 10:03:56 UTC
The problem is because of 'document.body.scrollTop' always being 0, ignoring the actual position of the scrollbar. I can only reproduce this problem with the webkit-2.2 branch.
Comment 5 Guillaume Desmottes 2013-09-16 10:21:49 UTC
This seems to be a WebKitGtk regression, I forwarded this bug to https://bugs.webkit.org/show_bug.cgi?id=121419
Comment 6 Guillaume Desmottes 2013-09-16 15:16:36 UTC
Ok actually that's something we can fix in Empathy.
Comment 7 Guillaume Desmottes 2013-09-16 15:19:30 UTC
Created attachment 255036 [details] [review]
Template.html: remove DOCTYPE

This ensures that WebKit considers the document in 'quirks' mode which is now
needed when using document.body.scrollTop (used to check if we should scroll
down or not when adding a new message).

See https://bugs.webkit.org/show_bug.cgi?id=121419 for details.
Comment 8 Guillaume Desmottes 2013-09-16 15:27:06 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.