add class to div wrapping Add New link [#416046]

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.

Comment	File	Size	Author
#1	add_class.patch	900 bytes	cYu

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

cYu CreditAttribution: cYu commented 31 March 2009 at 14:19

Status:

Active

» Needs review

File	Size
add_class.patch	900 bytes

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

Comment #2

zoo33 CreditAttribution: zoo33 commented 8 April 2009 at 08:04

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.

Comment #3

cYu CreditAttribution: cYu commented 8 April 2009 at 12:51

Status:

Active

» Needs work

Comment #4

cYu CreditAttribution: cYu commented 8 April 2009 at 12:58

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.