Since maintainer do not reply to all patches submitted I thought it will be good idea to post complete packages for someone like me looking for working modal_noderef module =)

I've combined several patches found in Issue list and reworked module myself to fit more into real lief and fix issues. You can find it attached. Here is list of things I remember I have changed:
- Fix last comma issue with IE (initial thanks to gleroux02 - http://drupal.org/node/764760 and latter incorporated same change into Edit part of the JS)
- Added edit link (initial thanks to semiaddict - http://drupal.org/node/713760)
- Edit link shown if you add new field for fields with "Unlimited" values (since it didn't went through form_alter hook, with original patch these links disappears once you add another field)
- Fix issue with IE blanking the whole screen when you close modal popup without submitting form
- Removed "Preview" button from node form in modal popup
- When press "Delete" button in modal popup - it silently deletes node, close popup and clear target field now. (probably we need to add some JS prompt("are you sure you want to delete this node?") there? )
- Probably something else, but do not remember already =)

Comments

pavel.karoukin’s picture

Status: Active » Needs review
pavel.karoukin’s picture

StatusFileSize
new9.92 KB

Some update. There were some issues with "Edit" link on adding new field. Use new version attached.

mattgilbert’s picture

awesome! Works great for me. Thanks hippich, semiaddict, and whomever else had a hand in this.

mbutcher’s picture

If there are any changes in this patch that were not part of other tickets, please roll them separately. I'm trying to get together a new release.