style(attachments): improve when we can download our multiple attachments (dossier en_construction)

app/assets/stylesheets/attachment.scss

@@ -21,3 +21,14 @@
     margin-top: 0.5rem;
   }
 }
+
+.attachment-multiple:not(.fr-downloads-group) {
+  ul {
+    list-style-type: none;
+    padding-inline-start: 0;
+  }
+
+  li {
+    padding-bottom: 0;
+  }
+}

app/assets/stylesheets/forms.scss

@@ -170,8 +170,7 @@
   input[type=number],
   input[type=tel],
   textarea,
-  select,
-  .attachment {
+  select {
     display: block;
     margin-bottom: $default-fields-spacer;
 

app/components/attachment/edit_component/edit_component.html.haml

@@ -1,4 +1,4 @@
-.fr-mb-2w.attachment.fr-upload-group{ id: attachment ? dom_id(attachment, :edit) : nil }
+.attachment.fr-upload-group{ id: attachment ? dom_id(attachment, :edit) : nil, class: class_names("fr-mb-2w": !(as_multiple? && downloadable?)) }
   - if persisted?
     %div{ id: dom_id(attachment, :persisted_row) }
       .flex.flex-gap-2{ class: class_names("attachment-error": attachment.virus_scanner_error?) }

app/components/attachment/multiple_component.rb

@@ -10,6 +10,7 @@ class Attachment::MultipleComponent < ApplicationComponent
   attr_reader :max
   attr_reader :user_can_destroy
   attr_reader :user_can_download
+  alias user_can_download? user_can_download
 
   delegate :count, :empty?, to: :attachments, prefix: true
 

app/components/attachment/multiple_component/multiple_component.html.haml

@@ -1,9 +1,10 @@
-.fr-mb-4w.attachment-multiple
+.fr-mb-4w.attachment-multiple{ class: class_names("fr-downloads-group": user_can_download?) }
   = template
 
-  - each_attachment do |attachment, index|
-    %div{ id: dom_id(attachment) }
-      = render Attachment::EditComponent.new(champ:, attached_file:, attachment:, index:, as_multiple: true, user_can_destroy:, user_can_download:)
+  %ul
+    - each_attachment do |attachment, index|
+      %li{ id: dom_id(attachment) }
+        = render Attachment::EditComponent.new(champ:, attached_file:, attachment:, index:, as_multiple: true, user_can_destroy:, user_can_download:)
 
   %div{ id: empty_component_id, class: class_names("hidden": !can_attach_next?) }
     = render Attachment::EditComponent.new(champ:, attached_file:, attachment: nil, index: attachments_count, user_can_destroy:)