Merge pull request #14037 from ncounter/fix-arrow-up

Set proper collapsed state on loading
openSUSE · Mar 20, 2023 · dd3872b · dd3872b
2 parents 5bc5d36 + 7a11caf
commit dd3872b
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/api/app/components/diff_list_component.html.haml b/src/api/app/components/diff_list_component.html.haml
@@ -1,15 +1,17 @@
 - diff_list.each_with_index do |(name, file_info), file_index|
   - state = file_info['state']
+  - expanded = expand?(name, state)
   .accordion.mb-2{ id: "diff-list-#{name.parameterize}" }
     .accordion-item
       %h2.accordion-header
         %button.accordion-button.text-bg-light{ type: 'button', data: { 'bs-toggle': 'collapse',
                                                                         'bs-target': "#diff-item-#{name.parameterize}" },
-                                                aria: { expanded: 'true', controls: "diff-item-#{name.parameterize}" } }
+                                                aria: { expanded: 'true', controls: "diff-item-#{name.parameterize}" },
+                                                class: expanded ? '' : 'collapsed' }
           - if (old_file = file_info['old'])
             = changed_filename(old_file['name'], name, state)
           - else
             = name
           %span.badge.ms-1{ class: badge_for_state(state) }= state.capitalize
-      .accordion-collapse.collapse{ class: expand?(name, state) ? 'show' : '', id: "diff-item-#{name.parameterize}", 'data-object': view_id }
+      .accordion-collapse.collapse{ class: expanded ? 'show' : '', id: "diff-item-#{name.parameterize}", 'data-object': view_id }
         = render(DiffComponent.new(diff: file_info.dig('diff', '_content'), file_index:, commentable:))