Only require reason when other category is selected

openSUSE · Oct 6, 2023 · 4f0c015 · 4f0c015
1 parent 8618735
commit 4f0c015
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/api/app/assets/javascripts/webui/report.js b/src/api/app/assets/javascripts/webui/report.js
@@ -42,6 +42,6 @@ function hideReportButton(element) {
 
 $(document).ready(function(){
   $('#report-category').on('change', '.form-check-input', function(e) {
-    $('#report-reason').toggleClass('d-none', ( e.target.value !== 'other' ));
+    $('#report-reason textarea').attr('required', (e.target.value !== 'other' ? null : true));
   });
 });
diff --git a/src/api/app/views/webui/shared/_report_modal.html.haml b/src/api/app/views/webui/shared/_report_modal.html.haml
@@ -23,7 +23,7 @@
                 .form-check
                   = radio.radio_button(class: 'form-check-input')
                   = radio.label(class: 'form-check-label')
-            .d-none#report-reason
+            #report-reason
               = form.label('Reason:')
               = form.text_area(:reason, class: 'form-control')
               .text-muted