Skip to main content

Search and Top Navigation

#4118 closed bug (fixed)

Opened February 11, 2009 12:19PM UTC

Closed February 13, 2009 03:48AM UTC

Sortable: option sortIndicator is undocumented

Reported by: rdworth Owned by: scottgonzalez
Priority: blocker Milestone: 1.7
Component: ui.sortable Version: 1.6rc6
Keywords: Cc:
Blocked by: Blocking:


Current default: ???

Current description: ???

Attachments (0)
Change History (3)

Changed February 12, 2009 10:25AM UTC by paul comment:1

I really would like to keep this undocumented. It's a bad pattern (it exposes the full internal API), but needed for now because the sortable tree uses it. I want to get rid of it in the next version, as soon as we have basic inheritance patterns available.

Changed February 13, 2009 03:43AM UTC by scottgonzalez comment:2

owner: → scott.gonzalez
status: newaccepted

Since the sortable tree isn't part of UI yet (not even in the dev branch, only in experimental), I don't think this is a valid reason to keep around an invalid option.

Changed February 13, 2009 03:48AM UTC by scottgonzalez comment:3

resolution: → fixed
status: acceptedclosed

Fixed in r2070. Removed option and updated docs.