Merge pull request #2471 from bgeuken/maintainers

[webui] Fix closing of package maintainer list
openSUSE · Dec 16, 2016 · a26cc44 · a26cc44
2 parents af51d52 + 684cb50
commit a26cc44
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/api/app/assets/javascripts/webui/application.js b/src/api/app/assets/javascripts/webui/application.js
@@ -255,8 +255,8 @@ $(function() {
   $('.show_dialog').on('click', function() {
     $($(this).data('target')).removeClass('hidden');
   });
+});
 
-  $('.close-dialog').on('click', function() {
-    $($(this).data('target')).addClass('hidden');
-  });
+$(document).on('click','.close-dialog', function() {
+  $($(this).data('target')).addClass('hidden');
 });
diff --git a/src/api/app/views/webui/request/_package_maintainers_dialog.html.erb b/src/api/app/views/webui/request/_package_maintainers_dialog.html.erb
@@ -15,7 +15,7 @@
          </table>
 
         <div class="dialog-buttons">
-          <%= link_to('close', '#', title: 'Close', class: 'close_dialog', data: { target: '#maintainers_dialog' }) %>
+          <%= link_to('close', '#', title: 'Close', class: 'close-dialog', data: { target: '#maintainers_dialog' }) %>
         </div>
       </div>
     </div>