make batch instruction button disable

app/components/dossiers/batch_operation_inline_buttons_component/batch_operation_inline_buttons_component.html.haml

@@ -1,5 +1,5 @@
 - if opt.keys.include?(:instruction)
-  = render Dropdown::MenuComponent.new(wrapper: :div, wrapper_options: { data: {controller: 'menu-button', popover: 'true', operation: opt[:operation]} }, menu_options: { id: "dropdown_batch" }, button_options: { class: "fr-btn fr-btn--sm fr-ml-1w"}, role: :region ) do |menu|
+  = render Dropdown::MenuComponent.new(wrapper: :div, wrapper_options: { data: {controller: 'menu-button', popover: 'true', operation: opt[:operation]} }, menu_options: { id: "dropdown_batch" }, button_options: { disabled: true, data: { batch_operation_target: "menu" }, class: "fr-btn fr-btn--sm fr-ml-1w"}, role: :region ) do |menu|
     - menu.with_button_inner_html do
       = t(".labels.instruction")
 

app/components/dropdown/menu_component.rb

@@ -51,4 +51,12 @@ class Dropdown::MenuComponent < ApplicationComponent
   def button_class_names
     ['fr-btn', 'dropdown-button'] + Array(@button_options[:class])
   end
+
+  def disabled?
+    @button_options[:disabled] == true
+  end
+
+  def data
+    { menu_button_target: 'button' }.deep_merge(@button_options[:data].to_h)
+  end
 end

app/components/dropdown/menu_component/menu_component.html.haml

@@ -1,5 +1,5 @@
 = content_tag(@wrapper, wrapper_options) do
-  %button{ class: button_class_names, id: button_id, data: { menu_button_target: 'button' }, "aria-expanded": "false", 'aria-haspopup': 'true', 'aria-controls': menu_id }
+  %button{ class: button_class_names, id: button_id, disabled: disabled?, data: data, "aria-expanded": "false", 'aria-haspopup': 'true', 'aria-controls': menu_id }
     = button_inner_html
 
   %div{ data: { menu_button_target: 'menu' }, id: menu_id, 'aria-labelledby': button_id, role: menu_role, 'tab-index': -1, class: menu_class_names }

app/javascript/controllers/batch_operation_controller.ts

@@ -5,7 +5,7 @@ import invariant from 'tiny-invariant';
 export class BatchOperationController extends ApplicationController {
   static targets = ['menu', 'input'];
 
-  declare readonly menuTarget: HTMLButtonElement;
+  declare readonly menuTargets: HTMLButtonElement[];
   declare readonly hasMenuTarget: boolean;
   declare readonly inputTargets: HTMLInputElement[];
 
@@ -78,16 +78,17 @@ export class BatchOperationController extends ApplicationController {
         switchButton(button, available);
         return available;
       });
+
       if (this.hasMenuTarget) {
         if (available.length) {
-          enable(this.menuTarget);
+          this.menuTargets.forEach((e) => enable(e));
         } else {
-          disable(this.menuTarget);
+          this.menuTargets.forEach((e) => disable(e));
         }
       }
     } else {
       if (this.hasMenuTarget) {
-        disable(this.menuTarget);
+        this.menuTargets.forEach((e) => disable(e));
       }
       buttons.forEach((button) => switchButton(button, false));
     }

spec/components/dossiers/batch_operation_component_spec.rb

@@ -21,7 +21,7 @@ RSpec.describe Dossiers::BatchOperationComponent, type: :component do
   context 'statut suivis' do
     let(:statut) { 'suivis' }
     it { is_expected.to have_button('Passer les dossiers en instruction', disabled: true) }
-    it { is_expected.to have_button('Accepter les dossiers', disabled: true) }
+    it { is_expected.to have_button('Instruire les dossiers', disabled: true) }
     it { is_expected.to have_button('Autres actions multiples', disabled: true) }
     it { is_expected.to have_button('Repasser les dossiers en construction', disabled: true) }
     it { is_expected.to have_button('Ne plus suivre les dossiers', disabled: true) }