Bug 1466450 - Remove margin and end padding from close button of animation inspector. r?daisuke draft
authorMantaroh Yoshinaga <mantaroh@gmail.com>
Tue, 05 Jun 2018 14:43:07 +0900
changeset 803972 1bdfa4608b99b5ca2d2d36c2e0591f6f3989ea1a
parent 803743 d8f180ab74921fd07a66d6868914a48e5f9ea797
push id112248
push userbmo:mantaroh@gmail.com
push dateTue, 05 Jun 2018 05:44:38 +0000
reviewersdaisuke
bugs1466450
milestone62.0a1
Bug 1466450 - Remove margin and end padding from close button of animation inspector. r?daisuke This patch will remove tab indent as well and make close button of animation inspector to be right aligned. MozReview-Commit-ID: GUr1LtsKuLX
devtools/client/themes/animation.css
--- a/devtools/client/themes/animation.css
+++ b/devtools/client/themes/animation.css
@@ -381,25 +381,36 @@ select.playback-rate-selector.devtools-b
   height: 100%;
   overflow: hidden;
   width: 100%;
   z-index: 1;
 }
 
 .animation-detail-header {
   display: flex;
+  padding-inline-end: 0;
+}
+
+/* On OSX the cursor turns into a window-resizing cursor at the edges of the
+ * window, so bring the end of the close button in. */
+:root[platform="mac"] .animation-detail-header {
+  padding-inline-end: 3px;
 }
 
 .animation-detail-title {
   flex: 1;
   overflow: hidden;
   text-overflow: ellipsis;
   white-space: nowrap;
 }
 
+.animation-detail-close-button {
+  margin: 0;
+}
+
 .animation-detail-close-button::before {
   background-image: url(chrome://devtools/skin/images/close.svg);
 }
 
 /* Animated Property List Container */
 .animated-property-list-container {
   display: flex;
   flex: 1;
@@ -482,28 +493,28 @@ select.playback-rate-selector.devtools-b
   width: 0;
 }
 
 /* Animated Property List */
 .animated-property-list-background {
   border-left: var(--tick-line-style);
   border-right: var(--tick-line-style);
   bottom: 0;
-	left: var(--sidebar-width);
+  left: var(--sidebar-width);
   min-height: 100%;
-	position: sticky;
+  position: sticky;
   top: 0;
-	width: calc(100% - var(--sidebar-width) - var(--graph-right-offset));
+  width: calc(100% - var(--sidebar-width) - var(--graph-right-offset));
 }
 
 .animated-property-list-background span {
   border-left: var(--tick-line-style);
-	height: 100%;
-	left: 50%;
-	position: absolute;
+  height: 100%;
+  left: 50%;
+  position: absolute;
 }
 
 .animated-property-list {
   flex: 1;
   list-style-type: none;
   margin: 0;
   padding: 0;
   position: absolute;