Download as
Browse Code »

Commit 8a6b4b70fc3b55cfe57eed858ef7d0a915174325

Authored by Larissa Reis
1 parent 0c14356c
Exists in master and in 22 other branches angular_poc, api-article-archived, article-readonly, backup-7-jan-2016, captcha_serpro_plugin, content-manager-hostspot, export-comment-paragraph, fix_event_date_issue, login-captcha, master_rails3, new_video_plugin, pg_search_rank, production, refactor_with_role, refactor_with_role_scope, remove_profile_cat_icons, resend_confirmation_email, staging, staging_rails3, tasks_keep_filter_params, theme-brasil-digital-from-staging, travis

Adds delete button to new product's qualifiers 

  Adds a delete button along with the new qualifiers row.
  Also, fix the page so when the qualifiers is selected, the button
  remains (this bug only happened in Chrome and was caused by a blank
  character in the html)

(ActionItem2374)
Showing 2 changed files with 4 additions and 4 deletions   Show diff stats
app/views/manage_products/_certifiers_for_selection.rhtml
  Diff comments   View file @8a6b4b7
1   -<%= select_certifiers(@qualifier) %>
2   -<%= button_to_function(:delete, '', "jQuery(this).parents('tr').remove()") %>
  1 +<%= select_certifiers(@qualifier) +
  2 + button_to_function(:delete, content_tag('span', 'Delete qualifier'), "jQuery(this).parents('tr').remove()") %>
3 3 <% javascript_tag do %>
4 4 jQuery('#product-qualifiers-list *').removeClass('small-loading')
5 5 <% end %>
... ...
app/views/manage_products/_edit_info.rhtml
  Diff comments   View file @8a6b4b7
... ... @@ -39,8 +39,8 @@
39 39 <%= select_qualifiers(@product, qualifier.id) %>
40 40 </td>
41 41 <td id='certifier-area-<%= index %>'>
42   - <%= select_certifiers(qualifier, @product) %>
43   - <%= button_to_function(:delete, content_tag('span', 'Delete qualifier'), "jQuery(this).parents('tr').remove()") %>
  42 + <%= select_certifiers(qualifier, @product) +
  43 + button_to_function(:delete, content_tag('span', 'Delete qualifier'), "jQuery(this).parents('tr').remove()") %>
44 44 </td>
45 45 </tr>
46 46 <% end %>
... ...