Skip to main content

Search and Top Navigation

#4305 closed bug (invalid)

Opened March 09, 2009 04:44PM UTC

Closed November 08, 2012 08:53AM UTC

Sortable: Revert placeholder calcing position

Reported by: Zebulon Owned by: Zebulon
Priority: minor Milestone: 2.0.0
Component: ui.sortable Version: 1.7
Keywords: Cc:
Blocked by: Blocking:
Description

The placeholder margins left & top are not included in the revert

position calcing causing a gap if margin(s) are defined.

Here's a fix ( works fine FF & Safari ) :

Line 288 - Function _mouseStop :

if(this.options.revert) {

var self = this;

var cur = self.placeholder.offset();

self.reverting = true;

'''var margin_left=parseInt(self.placeholder.css('marginLeft'),10);

var margin_top=parseInt(self.placeholder.css('marginTop'),10);

$(this.helper).animate({

left: cur.left - margin_left .......... ,

top: cur.top - margin_top ...........'''

}, parseInt(this.options.revert, 10) || 500, function() { self._clear(event); });

Attachments (0)
Change History (4)

Changed May 07, 2009 01:22PM UTC by jzaefferer comment:1

milestone: → 1.8

Changed October 11, 2012 02:54PM UTC by scottgonzalez comment:2

milestone: 1.9.02.0.0

Changed October 25, 2012 12:50AM UTC by petersendidit comment:3

owner: → Zebulon
status: newpending
summary: Revert placeholder calcing positionSortable: Revert placeholder calcing position

Thanks for taking the time to contribute to the jQuery UI project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket.

Additionally, be sure to test against the git version of both jQuery UI and jQuery to ensure the issue still exists. To get you started, use this boilerplate: http://jsfiddle.net/ZgAqH/Open the link and click to "Fork" (in the top menu) to get started.

Changed November 08, 2012 08:53AM UTC by trac-o-bot comment:4

resolution: → invalid
status: pendingclosed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!