The jQuery UI of the sort feature does not work with Rubik (seems to work fine with Garland, but I'm not using that for Admin theme). So I tried using the drag handle version of the sort feature and it doesn't save the sort. It even states after saving that "The order of the images was left unchanged".

As for what I mean when I say the JQuery UI doesn't work with Rubik, I mean that when I select an image to drag it disappears or other images disappear and there is no way to continue sorting as there is no visual indication of where any images are.

Thanks!

CommentFileSizeAuthor
#15 jquery conflicts.txt1.06 KBdddave

Comments

jon betts’s picture

jon betts commented
Title: Non JQuery UI srot not working » Non JQuery UI sort not working

Updated typo in title.

scroogie’s picture

scroogie commented
Category: bug » support
Priority: Major » Normal

Hi,

these are actually two seperate issues. First, the tabledrag doesn't work because there is a small mistake in Node Gallery. The patch is here: #1023968: Sort tabledrag broken

Second, Rubik/Tao does not interfere with the jquery ui sorting. I just tried that out and it's working flawlessly. Perhaps you have other modules that interfere, such as jquery_update, or you use a newer version of jquery_ui?

jon betts’s picture

jon betts commented

@scroogie, Thanks for looking into this! It was jquery_update! I disabled it and the sort is working fine. I don't think I _need_ to have the update so this works fine. I'll look into the patch as well; however the draggable sort is hard to beat UX-wise.

scroogie’s picture

scroogie commented

Thanks for the feedback. I would try to make it compatible with jquery_update, but as there are so many bugs in Drupal and other modules with it, I don't know if it merits the effort.

justintime’s picture

justintime commented
Title: Non JQuery UI sort not working » Make jQuery-based sort work with jquery_update, or alert the admin about incompatability
Category: support » task

Spoke with @scroogie about this a bit. We will either make our sort work with AND without jquery_update module (fairly unlikely), or alert the admin about the incompatability in admin/reports/status (much more likely).

scroogie’s picture

scroogie commented
Status: Active » Fixed

Mhm, weird. It actually works. I just tried it out with jquery_update 2.x and jquery_ui, which means jquery 1.3.2 and jqueryUI 1.7.3. No errors, sorting works as expected.

I assume that people with problems have a version of UI that doesn't fit to the version of jquery. For example using juery 1.3.2 with jquery UI 1.6 won't work. Unfortunately there is no way we can check that, or even solve problems with that.

mshepherd’s picture

mshepherd commented

I've also got it working with the combination at #6

jon betts’s picture

jon betts commented

I wasn't using jquery_update 2.x as it isn't a recommended release yet, so that probably has something to do with it. Thanks for looking into this further. It's good to know it works with a updated version.

scroogie’s picture

scroogie commented

glowsinthedark, just to clarify: you should never use a newer jquery UI without a corresponding jquery! They are specifically suited at one another.

jon betts’s picture

jon betts commented

scroogie, thanks! Is there any particular clue as to which version goes with which? For example, the two versions above have no version number parity and they aren't current with what is available. Just curious. It makes perfect sense that they would need to be at the same level, but figuring out the levels hasn't made sense to me yet.

justintime’s picture

justintime commented

From http://jqueryui.com/docs/Getting_Started

Step 3: Choose a version of jQuery UI
The last step in the download builder is to select a version number. This is a very important step because jQuery UI versions are designed to work with specific versions of jQuery. The current versions are: jQuery UI 1.8.9: Works with jQuery 1.3.2+ jQuery UI 1.7.2: Works with jQuery 1.3.2+

scroogie’s picture

scroogie commented

I always use the following as rule of thumb, although some minor versions are compatible to other minor versions.

jquery 1.2 -> jquery UI 1.6
jquery 1.3 -> jquery UI 1.7
jquery 1.4 -> jquery UI 1.8

jon betts’s picture

jon betts commented

@ justintime and @scroogie, thanks very much! This is very helpful and much appreciated!

Status: Fixed » Closed (fixed)

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

dddave’s picture

dddave commented
Component: Code » Documentation
Status: Closed (fixed) » Needs review
StatusFileSize
new jquery conflicts.txt1.06 KB

I've hammered together a short help text about this problem. Please review.

@justintime, scroogie
Could I simply put this .txt file into the dropbox folder or do I need to use a specific format?

edit: marked #1091202: Manual Sorting of images fails as a dupe.

scroogie’s picture

scroogie commented

The best is to use simple HTML formatting, e.g. h2 for headlines, p for paragraphs and ul and li for itemized lists.

I commited your help text to git with the missing HTML, you can take that or the other help files as an example.

Thanks!

dddave’s picture

dddave commented
Status: Needs review » Closed (fixed)

Thanks for committing and adding the html.

Re-closing the issue.