From 812cefb62db3f4b82705a9c07b6e294be2c292f8 Mon Sep 17 00:00:00 2001
From: mfluharty <mfluharty@gitlab.com>
Date: Sun, 15 Sep 2019 21:50:26 -0600
Subject: Improve pipeline CSS

Use utility class for unstyled list
Remove direct child requirement for highlighting expanded job
---
 .../pipelines/components/graph/stage_column_component.vue          | 7 ++++++-
 app/assets/stylesheets/pages/pipelines.scss                        | 6 +-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/app/assets/javascripts/pipelines/components/graph/stage_column_component.vue b/app/assets/javascripts/pipelines/components/graph/stage_column_component.vue
index e9a250effd4..c98f2e0de97 100644
--- a/app/assets/javascripts/pipelines/components/graph/stage_column_component.vue
+++ b/app/assets/javascripts/pipelines/components/graph/stage_column_component.vue
@@ -61,7 +61,12 @@ export default {
 
     <div class="builds-container">
       <ul>
-        <li v-for="group in groups" :id="groupId(group)" :key="group.id" class="build">
+        <li
+          v-for="group in groups"
+          :id="groupId(group)"
+          :key="group.id"
+          class="build list-unstyled"
+        >
           <job-item
             v-if="group.size === 1"
             :job="group.jobs[0]"
diff --git a/app/assets/stylesheets/pages/pipelines.scss b/app/assets/stylesheets/pages/pipelines.scss
index 5cb538ec03e..7a90777487a 100644
--- a/app/assets/stylesheets/pages/pipelines.scss
+++ b/app/assets/stylesheets/pages/pipelines.scss
@@ -494,10 +494,6 @@
         @include flat-connector-before;
       }
     }
-
-    li {
-      list-style: none;
-    }
   }
 
   .stage-name {
@@ -518,7 +514,7 @@
 
     .ci-job-dropdown-container {
       // highlight job whose dropdown is shown
-      &.dropdown.show > .build-content {
+      &.dropdown.show .build-content {
         background-color: $ci-action-dropdown-expanded-background;
 
         &:hover {
-- 
cgit v1.2.1