cdotyone / mochaui

development tree for MochaUI
http://mochaui.org/demo/
Other
290 stars 83 forks source link

collapse panels in demo (demo/index.html) croaks: MUI.get() on panel.id which is "" #113

Closed GerHobbelt closed 12 years ago

GerHobbelt commented 13 years ago

cause: mocha has a Sortables for columns: this one clones the DOM on start (click) and the clone gets all its 'id' attributes wiped (mootools: cleanClone()). The regular click handler (when clicking on the collapse/expand buttons in the panel toolbars) apparently delivers these clones in the 'panel' variables, thus breaking the code in several places.

A temporary 'workaround' is to set Controls/column.js options.sortable to FALSE. (losing you the sortable ~ reordering functionality for panels in a column)

GerHobbelt commented 13 years ago

:-( demo/index.html is not in HEAD; a fork/merge mismatch on my side. Nevertheless, went and tested with the 'official' Demo/demo-application-html.html and same issue (and work-around) there.

FF4 / Safari 5 on Win7/64

cdotyone commented 12 years ago

As you stated the drag events where cloning while collapsing/expanding. I took care of this in previous path within the last week. It is in the most current develop branch. -Chris