#4438 closed bug (wontfix)
Helper is in the wrong position when body has position: relative and an offset relative to the document
Reported by: | FernandoFerreira | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | ui.selectable | Version: | 1.7.1 |
Keywords: | selectable, helper | Cc: | |
Blocked by: | Blocking: |
Description
Attached to this bug report there is a minimal testcase to illustrate what happens when:
- The body has position: relative
- There is an offset between the body top (or left) and the document top (or left)
The abnormal behaviour is in the helper, whose position is offset by the same ammount the body is, in this example meaning 70px in each direction (50px because of the html padding, and 20px because of the body margin).
Drag the mouse cursor inside the green body to illustrate the bug.
What causes this bug is the assumption that origin point of the body (0,0) will be the same of the document, making the absolute coordinates of the helper relative to the body coincide with the one relative to the document. That happens to be true if the body position is static, delegating the origin point (0,0) to the document.
That ceases to be true when the body position is set to relative, triggering the condition for this bug to appear.
One possible solution is to discount the body offset in the value of pageX and pageY, the the lines [source:/trunk/ui/[email protected]#L67 67] and [source:/trunk/ui/[email protected]#L134 134] of the ui.selectable.js file, like this:
this.opos = [event.pageX - document.body.getClientRects()[0].left - (document.documentElement.scrollLeft || document.body.scrollLeft), event.pageY - document.body.getClientRects()[0].top - (document.documentElement.scrollTop || document.body.scrollTop)];
and
var x1 = this.opos[0], y1 = this.opos[1], x2 = event.pageX - document.body.getClientRects()[0].left - (document.documentElement.scrollLeft || document.body.scrollLeft), y2 = event.pageY - document.body.getClientRects()[0].top - (document.documentElement.scrollTop || document.body.scrollTop);
Unfortunetaly, document.body.getClientRects() doesn't work on webkit based browsers, so this is probably not the right solution.
Attachments (1)
Change History (6)
Changed 13 years ago by
Attachment: | index.html added |
---|
comment:1 Changed 13 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
jQuery doesn't support dimensions-related methods when the body is positioned. You should use a wrapper div instead as a workaround.
comment:4 Changed 13 years ago by
My body is at (0,0), but the div I am specifying as my selectable is not at 0,0, and the lasso rectangle is off (from the mouse) by the offset of the div from 0,0. What is the work-around to this? Is this a bug?
Minmal testcase to illustrate the bug