Mercurial > defr > drupal > noderef_view
changeset 7:281e131cb6c3
Change location of the out-of-space warning and way to pick the wrapper
Previously the warning was screwed when the user pressed the "Add more" button
(the possibilites were: warning above the label which looked strange, warning
below the the add more button but that would then expand to contain the whole
field or warning between the textfields and the add more button but the
warning would disaspear when the button was clicked). It's now conveniently
placed under the "Search" link.
A class is also added to the <div /> wrapping the textfields, so we use that
to find the wrapper instead of walking the DOM tree.
| author | Franck Deroche <franck@defr.org> |
|---|---|
| date | Tue, 27 Jan 2009 11:54:25 +0100 |
| parents | e3f85c9247b7 |
| children | 1f9c278126f9 |
| files | noderef_view.js |
| diffstat | 1 files changed, 11 insertions(+), 6 deletions(-) [+] |
line wrap: on
line diff
--- a/noderef_view.js Mon Jan 26 15:09:55 2009 +0100 +++ b/noderef_view.js Tue Jan 27 11:54:25 2009 +0100 @@ -7,13 +7,14 @@ $('.noderef_view_link', context).each(function(i, obj) { $(obj).addClass('noderef_view_link_processed').click(function() { // Get the wrapper - var wrapper = $(this).prev(); - while (!wrapper.is('div')) { - wrapper = wrapper.prev(); - } + var wrapper = $(this) + .parent() + .find('.noderef-view-items') + .filter(':first'); + var element = $(this); // Hide existing "No space left" message - $('div.noderef_nospace', wrapper).fadeOut(); + $('div.noderef_outofspace', element.parent()).fadeOut(); // Suppress behavior if it was previsously attached $(document).unbind('popups_form_success.noderef_view'); @@ -42,7 +43,11 @@ if (left.length) { message = Drupal.t('No space left for value(s): @titles', { '@titles': left.join(', ')}); - wrapper.append('<div class="warning noderef_nospace">'+ message +'</div>'); + text = $('<br /><div>'+ message + '<div>') + .css('display', 'inline') + .addClass('warning') + .addClass('noderef_outofspace'); + element.after(text); } }); });