The suffix that is added to nodereference form elements:

<div>Add New: ' . implode(', ', $links) .'</div>...

could benefit from having a class or id added to the div so that it can more easily be found with css or a jquery selector. My use case deals with removing this link once a node has been added and making it visible again if the nodereference autocomplete field is emptied.

CommentFileSizeAuthor
#1 add_class.patch900 bytescyu

Comments

cyu’s picture

cyu commented
Status: Active » Needs review
StatusFileSize
new add_class.patch900 bytes

It would also be nice to add this for styling purposes.

zoo33’s picture

zoo33 commented
Category: feature » bug
Status: Needs review » Active

In fact, that markup should be run through a theme function. There is now way right now to theme the output. And the text "Add new" needs to be wrapped in a t() function so that it can be translated. Forgive me for being blunt, but this is a bug.

cyu’s picture

cyu commented
Status: Active » Needs work
cyu’s picture

cyu commented
Category: bug » feature
Status: Needs work » Needs review

zoo33: Can you open a separate bug report for those things and add in that a t() needs added in the _popups_reference_links function? The issues you mention don't really have anything to do with this feature request.

hitfactory’s picture

hitfactory commented

Confirming the patch works for me.