Fix disabling of mark as read button

The button on the notification page was disabled through
a CSS class, which only lead to a disabled look, but still
allowed to submit the form.

Fixes #10248

Co-authored-by: Eduardo Navarro <enavarro@suse.com>

src/api/app/assets/javascripts/webui/notification.js

@@ -14,10 +14,10 @@ function setCheckboxCounterAndSubmitButton() {
   });
 
   if(amountBoxesChecked <= 0) {
-    $('#done-button').addClass('disabled');
+    $('#done-button').prop('disabled', true);
     $('#select-all-label').text('Select all');
   } else {
-    $('#done-button').removeClass('disabled');
+    $('#done-button').prop('disabled', false);
     $('#select-all-label').text(amountBoxesChecked + ' selected');
   }
 }

src/api/app/views/webui/users/notifications/_notification_action_bar.html.haml

@@ -6,5 +6,5 @@
           = check_box_tag('select-all-notifications', 1, false, class: 'custom-control-input')
           %label.custom-control-label.align-middle.mr-4#select-all-label{ for: 'select-all-notifications' } Select All
           - button_text = type == 'read' ? "Mark as 'Unread'" : "Mark as 'Read'"
-          = button_tag(type: 'submit', class: 'btn btn-sm btn-outline-success px-3 disabled', id: 'done-button') do
+          = button_tag(type: 'submit', class: 'btn btn-sm btn-outline-success px-3', id: 'done-button', disabled: 'disabled') do
             = button_text