At the moment one can search for, create and directly reference nodes from the nodereference widget in a modal frame, but if a user needs to make a quick change to a referenced node while editing the parent, they have to first view the node in the external window then edit manually. It'd save a lot of time if one could simply have a further operation exposed to the widget to edit a referenced node in the modal frame. One use case is when adding new nodes from the modal frame and I've forgotten to add a small detail in the node. This would allow me to make the change with minimal user navigation, and without breaking the workflow off adding or editing nodes from the parent window.

Downsides I can see are UI clutter with too many icons in the widget, and problematic integration with the current tabs in the modal frame.

Any thoughts?

Comments

markus_petrux’s picture

Something like this would have to be optional, so it also adds clutter to the Nodereference extras tab in the administration section.

I do not plan to do add this feature myself, though I'm open to review patches as far as this is optional and it is well resolved in the modal frame as well as in the admin page.

On the other hand, the node edit form is just at a single click from there, via view referenced node in new window. Not too bad. So I'm not sure if it really worths the effort to implement such a feature.

markus_petrux’s picture

Status: Active » Closed (won't fix)

I marking this as won't fix, but just because I do not plan to do it. Feel free to re-open if you want to work on a patch as described in #1, however.

jvandooren’s picture

Status: Closed (won't fix) » Needs work
StatusFileSize
new4.4 KB

Hello

I needed this functionality too and got started on something.

This patch changes the functionality behind the "view" button. It will not open the node view in a new tab/window. It will now open the edit node view in a modalbox. You can change what you want in the node, and click save. The modalbox will close and the reference field will be updated. I needed this because I'm using Automatic Nodetitles based on editable CCK fields in my node.

Now, I didn't create this patch for general use. This patch only fits my needs.

I only published this so people can build something decent, maybe starting from my functionality.

jvandooren’s picture

StatusFileSize
new4.4 KB

Due to a small bug in my previous patch:

markus_petrux’s picture

Added edit icon to sprite and CSS updated for the new edit button.
-- Patch: http://drupal.org/cvs?commit=367626
-- Sprite: http://drupalcode.org/viewvc/drupal/contributions/modules/noderelationsh...

markus_petrux’s picture

Status: Needs work » Fixed

Ok, committed to CVS.

http://drupal.org/cvs?commit=367644

Thanks!

Ozeuss’s picture

Title: edit referenced node in modal frame » edit referenced node in modal frame - call pre-render function?
StatusFileSize
new1.01 KB

I've noticed that in the last patch, _noderelationships_child_edit_form_alter(&$form) doesn't call the pre-render function like the "create" form.
I guess that we don't to show tabs for edit form (since it doesn't make alot of sense to show both "create" and "edit" tab), But what I miss is the page prefix and suffix (which inserts the div). Adding the pre-render call makes the UI more consistent because of this.

I added a simple patch.

Ozeuss’s picture

Status: Fixed » Needs review
markus_petrux’s picture

Status: Needs review » Fixed

Good catch. Thanks!

Patch committed, but also changed the pre-render callback code to avoid PHP notices, because the tabs are only rendered for "Search and reference" and/or "Create and reference". For "Edit referenced node" we do not render tabs.

http://drupal.org/cvs?commit=374356

Status: Fixed » Closed (fixed)

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