In case of the d7ux use of this module (or in any other case, when the dialog would be used more like a navigable lightbox), drag-and-drop functionality is not desired and we can make the output slimmer (and less potential for errors and UI confusion) if we don't use the draggable component.

Not sure if we can just remove this like I did, since I did not find any code in the parent.js to make the dialog draggable, but jQuery UI docs say the draggable component is optional.

(This is a total a pseudo-patch to show the area affected not an actual code suggestion).

CommentFileSizeAuthor
draggable.patch710 bytesgábor hojtsy

Comments

markus_petrux’s picture

markus_petrux commented

I just added a new option (draggable) which is enabled by default.

Commited to CVS: patch for D7.

Keeping this as "needs work" because now, if disabled, we still have a dialog title with cursor:move. hmm...

markus_petrux’s picture

markus_petrux commented
Status: Needs work » Fixed

Ok, this is now fixed for both, D6 and D7.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.