Project

General

Profile

Actions

Bug #1283

closed

Murm autoscrolls message ignoring attached picture

Added by krigstask about 11 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Plugins: Azoth
Target version:
Start date:
09/22/2013
Due date:
% Done:

100%

Estimated time:
5:00 h
Reported in:
master

Description

Upon arrival of new message with image attached, Murm autoscrolls chat window taking only text into account. That leaves image mostly hidden and forces user to scroll manually.

Actions #1

Updated by 0xd34df00d about 11 years ago

That's because autoscrolling happens only on DOMNodeInserted and window.onresize events, and image loading takes some time after DOMNodeInserted.

Give me the name of a JS DOM event that occurs on document height change and the fix will be a one-liner.

Actions #2

Updated by krigstask about 11 years ago

Why `window.onresize` doesn't catch it, then?

Actions #3

Updated by 0xd34df00d about 11 years ago

Because it's fired when viewport is resized (when you change LC window size, for example), not when document height is changed. I'd expect something like document.onresize instead, but there is no such event.

Actions #4

Updated by 0xd34df00d about 11 years ago

  • Status changed from New to Assigned
  • Target version set to 0.6.60
Actions #5

Updated by 0xd34df00d about 11 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 100

Applied in changeset main|commit:84c3432348fa22dc19b7ca6f99f9752e01c5db97.

Actions #6

Updated by 0xd34df00d about 11 years ago

  • Estimated time set to 5:00 h
Actions #7

Updated by 0xd34df00d about 11 years ago

  • Status changed from Resolved to Closed
Actions #8

Updated by krigstask over 9 years ago

Looks like the issue is back.

LeechCraft 0.6.70-2726-gea9f29a
Built with Qt 4.8.6, running with Qt 4.8.6
Running on: Mac OS X 10.9

Actually it's Mac OS X 10.10.2, by the way.

Actions #9

Updated by 0xd34df00d over 9 years ago

This is probably a slightly different issue (though with the same observed effect). Reported as #1947.

Actions

Also available in: Atom PDF