Tickets :: [#9458] Imp scrollbar doesn't work if dimpmain is not at the top

6.0.0-beta6

4/10/26

Summary	Imp scrollbar doesn't work if dimpmain is not at the top
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	goncalo.queiros (at) portugalmail (dot) net
Created	12/20/10 (5590 days ago)
Due
Updated	12/20/10 (5590 days ago)
Assigned
Resolved	12/20/10 (5590 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

12/20/2010 11:25:55 PM	Michael Slusarz	Comment #6	Reply to this comment
This fixed the horizontal layout (thanks), but the vertical has the same problem ;) Sorry for not mentioned it before, but just noticed now Fixed.

12/20/2010 11:25:45 PM	Git Commit	Comment #5	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9458~~: Adjust width based on viewport position of viewport content element http://git.horde.org/horde-git/-/commit/76f52c85a7bc3427dc6039f120fbcd29c7914fbe

12/20/2010 08:39:46 PM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #4	Reply to this comment
This fixed the horizontal layout (thanks), but the vertical has the same problem ;) Sorry for not mentioned it before, but just noticed now

12/20/2010 07:26:32 PM	Michael Slusarz	Comment #3 Assigned to Michael Slusarz State ⇒ Resolved	Reply to this comment
It is because viewport.js was using the wrong location method. Everything for dragdrop2 is now in relation to the viewport offset instead of relative to a particular element.

12/20/2010 07:25:13 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: ~~Bug #9458~~: Fix position determination of dragbar http://git.horde.org/horde-git/-/commit/cadf1032428dc4a3b4a3a71e64feb18a32505c67

12/20/2010 07:16:24 PM	goncalo (dot) queiros (at) portugalmail (dot) net	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Imp scrollbar doesn't work if dimpmain is not at the top Queue ⇒ IMP Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
If the "dimpmain" div is not at the top (offset = 70px ex), the scrollbar will not work properly. You drag it until only one message is seen, you drop, and the viewport will have +/- 4 messages..the same happens if you pull it down. This is happening on viewposrt.js line "l = parseInt((y - sp.pos) / sp.lh);", because "y" value is wrong, so the number of lines are badly calculated. I think this is related with recent dragdorp2.js changes, but can't figure out where.