Current HEAD from git tested and works without issues with newer jQuery UI 1.7.3 and Modal Frame 6.x-1.7.
Patch attached.

CommentFileSizeAuthor
#4 hostmaster_818900.patch1.61 KBdrewish
#1 hostmaster.diff.txt1.6 KBomega8cc
hostmaster.diff806 bytesomega8cc

Comments

omega8cc’s picture

omega8cc commented
StatusFileSize
new hostmaster.diff.txt1.6 KB

Ooops. Newer version of Modal Frame API introduces another requirement: http://drupal.org/project/jquery_update - patch updated.

omega8cc’s picture

omega8cc commented

Unfortunately the jquery_update module breaks signup form and add site form (ajax doesn't work), so probably the upgrade doesn't make any sense, at least now.

drewish’s picture

drewish commented
Status: Active » Needs review

The patch works for fetching the modules but modalframe_requirements() is reporting a failure. I'm not sure if there's something we can do about it.

drewish’s picture

drewish commented
StatusFileSize
new hostmaster_818900.patch1.61 KB

Okay posted a patch for modalframe #798618: Deprecate support for jQuery UI library 1.6.x. Updating the .make file to account for it.

adrian’s picture

adrian commented
Status: Needs review » Postponed

There's only an alpha release of jquery_update, and the modalframe module needs a patch applied during drush make to work.

i'm going to set this to postponed until those issues are resolved.

sfyn’s picture

sfyn commented

I have been using current HEAD on a 5.3 server all day and have encountered no problems with modal frame and jquery so far :

I was able to
- create platforms
- create, disable, verify, delete, enable and clone sites

Versions:
PHP: PHP 5.3.2-1ubuntu4.2 with Suhosin-Patch (cli) (built: May 13 2010 20:01:00)
Jquery_UI (drupal module): jquery_ui.info,v 1.3 2009/05/21 22:03:30 robloach
Jquery_UI: 1.6
Modalframe: modalframe.info,v 1.1.2.5 2010/01/01 20:31:01 markuspetrux Exp $

sfyn’s picture

sfyn commented
anarcat’s picture

anarcat commented

I tried this again - the problem is that jquery_ui.module 1.4 depends on jquery_ui 1.6, so it's incompatible with the latest modal frame. A bug should be submitted in jquery_ui.module to fix this.

steven jones’s picture

steven jones commented
Status: Postponed » Closed (won't fix)

I think we'll stick with the current versions for now, and punt any upgrades to 7.x-2.x.