Opened 11 years ago

Closed 7 years ago

Last modified 7 years ago

#4485 closed bug (fixed)

Sortable: Portlet demo is buggy

Reported by: verifier Owned by:
Priority: minor Milestone: 1.10.2
Component: ui.sortable Version: 1.7.1
Keywords: Cc:
Blocked by: Blocking:

Description

I've just tried the portlet demo: http://jqueryui.com/demos/sortable/#portlets

  1. Drag the first portlet in the last column to below all portlets in the first column.
  1. That will make the second portlet in the last column move to the third position (and therefore also produce one empty block in the second column) in the middle column.

The bug is #2, since I haven't touched that portlet, was just doing #1. After that, it's impossible to move anything to the third column.

Attachments (1)

portlets.png (24.2 KB) - added by verifier 11 years ago.
Screenshot

Download all attachments as: .zip

Change History (9)

Changed 11 years ago by verifier

Attachment: portlets.png added

Screenshot

comment:1 Changed 10 years ago by Jörn Zaefferer

Milestone: TBD1.7.2

comment:2 Changed 10 years ago by Jörn Zaefferer

Milestone: 1.7.21.8

comment:3 Changed 8 years ago by silesianlordt

Last edited 8 years ago by silesianlordt (previous) (diff)

comment:4 Changed 7 years ago by Scott González

Milestone: 1.9.02.0.0

comment:5 Changed 7 years ago by petersendidit

Status: newopen
Summary: Portlet demo is buggySortable: Portlet demo is buggy

comment:6 Changed 7 years ago by tj.vantoll

I would say that the plugin itself is working fine in the demo. Since the columns are floating they're going to clear when they run out of width.

I think a better showcase of the plugin would be to ditch the columns - http://jsfiddle.net/tj_vantoll/ERSzD/. That example works great in WebKit but is buggy in Firefox and Opera.

For now I say we put a min-width of 520px on the body - https://github.com/jquery/jquery-ui/pull/927. Thoughts?

comment:7 Changed 7 years ago by TJ VanToll

Resolution: fixed
Status: openclosed

Sortable: Adding a min-width to the body on the portlet demo. Fixed #4485: Portlet demo is buggy.

Changeset: 8655d91ac68538c37da4d94ecc287df8c90a96ae

comment:8 Changed 7 years ago by Scott González

Milestone: 2.0.01.10.2
Note: See TracTickets for help on using tickets.