Skip to main content

Search and Top Navigation

#5545 closed bug (fixed)

Opened April 24, 2010 08:15PM UTC

Closed November 13, 2012 02:43AM UTC

Last modified November 13, 2012 02:45AM UTC

Resizable: Callbacks ignore the grid

Reported by: nicholashall Owned by:
Priority: minor Milestone: 1.10.0
Component: ui.resizable Version: 1.8
Keywords: haspatch Cc:
Blocked by: Blocking:

If a resizable has a grid defined, the 'resize' callback continuously fires as though the box were resizing normally, one pixel at a time. The result is an erroneous and largely useless callback, essentially representing only user mouse movement.

Also from what I can tell the same logic is causing the ui.size.width argument that's passed to the 'stop' callback to be erroneous, having the value the box would have had if it were not constrained to a grid, instead of its true final width.

Attachments (0)
Change History (5)

Changed July 10, 2012 07:49PM UTC by eromba comment:1

Here is a test case:

I've submitted a pull request with a fix and unit test here:

Changed October 11, 2012 09:07PM UTC by scottgonzalez comment:2

milestone: TBD2.0.0

Changed October 26, 2012 01:18AM UTC by tj.vantoll comment:3

keywords: → haspatch
status: newopen
summary: Callbacks ignore the grid.Resizable: Callbacks ignore the grid

Changed November 13, 2012 02:43AM UTC by Ethan Romba comment:4

resolution: → fixed
status: openclosed

Resizable: Update CSS dimensions selectively. Fixes #7605 - Setting width and height when only one is changing

Resizable: Trigger resize event only when element is resized. Fixes #5545 - Callbacks ignore the grid.

Resizable: Added event tests. Fixes #5817 - resize event reports unconstrained ui.size

Changeset: 3974b55ba5078799df818c78d9273e11d9796ff3

Changed November 13, 2012 02:45AM UTC by mikesherov comment:5