Browse Source

feat(issues): Cleanup group details global styles (#79241)

Scott Cooper 4 months ago
parent
commit
bbf91bdcf1

+ 0 - 1
static/app/views/issueDetails/shortIdBreadcrumb.tsx

@@ -57,7 +57,6 @@ export function ShortIdBreadcrumb({
       />
       <ShortIdCopyable>
         <Tooltip
-          className="help-link"
           title={t(
             'This identifier is unique across your organization, and can be used to reference an issue in various places, like commit messages.'
           )}

+ 3 - 123
static/less/group-detail.less

@@ -5,7 +5,6 @@
 
 .group-detail {
   position: relative;
-  clear: both;
 
   h3 {
     font-size: 22px;
@@ -23,30 +22,9 @@
   }
 
   .count {
-    &.short-id-box {
-      min-width: 0;
-
-      span:not(.help-link) {
-        font-size: 18px;
-        line-height: 1.45;
-        display: block;
-        overflow: hidden;
-        text-overflow: ellipsis;
-        white-space: nowrap;
-        direction: rtl;
-      }
-    }
-
-    span:not(.help-link) {
-      font-size: 22px;
-      display: inline-block;
-      line-height: 1.2;
-    }
-  }
-
-  .assigned-to {
-    text-align: right;
-    position: relative;
+    font-size: 22px;
+    display: inline-block;
+    line-height: 1.2;
   }
 
   .meta {
@@ -54,36 +32,12 @@
     color: #949ea4;
   }
 
-  .id-label {
-    background: #fff;
-    max-width: 200px;
-    white-space: nowrap;
-    overflow: hidden;
-    text-overflow: ellipsis;
-  }
-
   .severity {
     display: inline-block;
     position: relative;
     top: -7px;
     padding: 0 10px;
   }
-
-  .nav-tabs {
-    margin-top: 20px;
-    margin-bottom: 0;
-  }
-
-  &:before {
-    position: absolute;
-    display: block;
-    content: '';
-    bottom: -1px;
-    left: -40px;
-    right: -40px;
-    height: 1px;
-    background: @trim;
-  }
 }
 
 /**
@@ -105,17 +59,6 @@
   .primary {
     flex: 1;
 
-    .user-report {
-      border-radius: 0;
-      box-shadow: none;
-      padding: 20px 30px 0 40px;
-      border: 0;
-      border-top: 1px solid lighten(@trim, 4);
-      margin: 0;
-
-      padding-bottom: 0;
-    }
-
     @media (min-width: 1200px) {
       // TODO(dcramer): whatever flex is doing here is causing this to extend to
       // whatever it pleases (e.g. 4k pixels on my 1280 screen)
@@ -420,19 +363,6 @@ div.traceback > ul {
       margin: 0 2px;
     }
 
-    .blame {
-      color: lighten(@gray, 5);
-
-      a {
-        color: @gray;
-      }
-
-      .icon-mark-github {
-        position: relative;
-        top: 1px;
-      }
-    }
-
     .tooltip-inner {
       word-wrap: break-word;
       text-align: left;
@@ -825,20 +755,6 @@ span.val {
             color: @gray-dark;
           }
         }
-
-        .back-to,
-        .pull-right a {
-          font-size: 16px;
-        }
-
-        .back-to {
-          border-left: 1px solid @trim;
-          padding: 2px 10px;
-          display: inline-block;
-          margin-left: 7px;
-          position: relative;
-          top: -3px;
-        }
       }
     }
   }
@@ -1001,17 +917,6 @@ ul.crumbs {
   }
 }
 
-/**
-* Responsive medium screen
-* ============================================================================
-*/
-
-@media (max-width: 991px) {
-  .group-stats-column {
-    float: none;
-  }
-}
-
 /**
 * Responsive small screen
 * ============================================================================
@@ -1066,10 +971,6 @@ ul.crumbs {
         }
       }
 
-      .user-report {
-        padding: 15px 0;
-      }
-
       .exception {
         margin: 0;
         padding: 15px 0;
@@ -1092,26 +993,5 @@ ul.crumbs {
       border: 0;
       margin-bottom: 20px;
     }
-
-    .detailed-error {
-      border-top: 1px solid @trim;
-    }
-  }
-
-  // Context callout
-
-  .context-summary {
-    flex-direction: column;
-    margin-top: 0;
-    padding: 0;
-    margin-bottom: 20px;
-  }
-
-  .context {
-    overflow: auto;
-
-    li {
-      width: 800px;
-    }
   }
 }

+ 0 - 14
static/less/layout.less

@@ -75,20 +75,6 @@ body.narrow {
           color: @gray-dark;
         }
       }
-
-      .back-to,
-      .pull-right a {
-        font-size: 16px;
-      }
-
-      .back-to {
-        border-left: 1px solid @trim;
-        padding: 2px 10px;
-        display: inline-block;
-        margin-left: 7px;
-        position: relative;
-        top: -3px;
-      }
     }
 
     .with-padding {