Use accordions for lists on the pulse page

Also display lists only if data is present
openSUSE · Dec 19, 2018 · 7bb83d8 · 7bb83d8
1 parent 173652f
commit 7bb83d8
Showing 1 changed file with 68 additions and 41 deletions.
diff --git a/src/api/app/views/webui2/webui/projects/pulse/_pulse_list.html.haml b/src/api/app/views/webui2/webui/projects/pulse/_pulse_list.html.haml
@@ -45,48 +45,75 @@
             There have been no requests send to this project.
 .row
   .col
-    %h5#pulse-packages
-      Package Changes
-    %hr
-    %ul.list-unstyled
-      - (commits + new_packages + deleted_packages + updates).sort_by(&:datetime).reverse_each do |log_entry|
-        %li
-          = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
+    - package_changes = commits + new_packages + deleted_packages + updates
+    - if package_changes.present?
+      %h5#pulse-packages
+        = link_to('.collapse-package-changes', aria: { controls: 'collapse-package-changes', expanded: 'true' },
+                  'data-toggle' => 'collapse', role: 'button') do
+          Package Changes
+          %small
+            %span.collapser (Hide)
+            %span.expander (Show)
+      %hr
+      .collapse.show.collapse-package-changes
+        %ul.list-unstyled
+          - package_changes.sort_by(&:datetime).reverse_each do |log_entry|
+            %li
+              = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
 
-    %h5#pulse-project
-      Project Changes
-    %hr
-    %ul.list-unstyled
-      - project_changes.each do |log_entry|
-        %li
-          = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
+    - if project_changes.present?
+      %h5#pulse-project
+        = link_to('.collapse-project-changes', aria: { controls: 'collapse-project-changes', expanded: 'true' },
+                  'data-toggle' => 'collapse', role: 'button') do
+          Project Changes
+          %small
+            %span.collapser (Hide)
+            %span.expander (Show)
+      %hr
+      .collapse.show.collapse-project-changes
+        %ul.list-unstyled
+          - project_changes.each do |log_entry|
+            %li
+              = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
 
-    %h5#pulse-builds
-      Builds
-      %small
-        (in the last 24 hours)
-    %hr
-    %ul.list-unstyled
-      - builds.each do |log_entry|
-        %li
-          = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
+    - if builds.present?
+      %h5#pulse-builds
+        = link_to('.collapse-builds', aria: { controls: 'collapse-builds', expanded: 'true' }, 'data-toggle' => 'collapse', role: 'button') do
+          Builds
+          %small
+            (in the last 24 hours)
+            %span.collapser (Hide)
+            %span.expander (Show)
+      %hr
+      .collapse.show.collapse-builds
+        %ul.list-unstyled
+          - builds.each do |log_entry|
+            %li
+              = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
 
-    %h5#pulse-collaboration
-      Collaboration
-    %hr
-    %ul.list-unstyled
-      - (branches + comments).sort_by(&:datetime).reverse_each.each do |log_entry|
-        %li
-          = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
-
-    %h5#pulse-requests
-      Requests
-      - if requests.count > 30
-        %small.text-muted
-          ( last 30,
-          = link_to(project_requests_path(project)) do
-            view all
-          )
-    %hr
-    = render partial: 'pulse_list_requests', locals: { requests: requests }
+    - collaboration = branches + comments
+    - if collaboration.present?
+      %h5#pulse-collaboration
+        = link_to('.collapse-collaboration', aria: { controls: 'collapse-collaboration', expanded: 'true' },
+                  'data-toggle' => 'collapse', role: 'button') do
+          Collaboration
+          %small
+            %span.collapser (Hide)
+            %span.expander (Show)
+      %hr
+      .collapse.show.collapse-collaboration
+        %ul.list-unstyled
+          - collaboration.sort_by(&:datetime).reverse_each.each do |log_entry|
+            %li
+              = render partial: 'pulse_list_entry', locals: { log_entry: log_entry, project: project }
 
+    - if requests.present?
+      %h5#pulse-requests
+        = link_to('.collapse-requests', aria: { controls: 'collapse-requests', expanded: 'true' }, 'data-toggle' => 'collapse', role: 'button') do
+          Requests
+          %small
+            %span.collapser (Hide)
+            %span.expander (Show)
+      %hr
+      .collapse.show.collapse-requests
+        = render partial: 'pulse_list_requests', locals: { requests: requests }