Changeset 190640 in webkit

Timestamp:

Oct 6, 2015, 1:28:25 PM (10 years ago)

Author:

commit-queue@webkit.org

Message:

Web Inspector: Fix spacing of FindBanner prev/next buttons
​https://bugs.webkit.org/show_bug.cgi?id=149717

Patch by Devin Rousso <Devin Rousso> on 2015-10-06
Reviewed by Darin Adler.

The sizing and spacing of the previous and next buttons in the FindBanner
is slightly off and should be readjusted to ensure that both buttons are
the same size and have the same spacing.

• UserInterface/Views/FindBanner.css:

(.find-banner > button.segmented.right > .glyph):
(.find-banner > button.segmented):
(.find-banner > button.segmented > .glyph):
(.find-banner > button.segmented.left): Deleted.
(.find-banner > button.segmented.left > .glyph): Deleted.
(.find-banner > button.segmented.right): Deleted.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/FindBanner.css (modified) (3 diffs)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2015-10-06  Devin Rousso  <dcrousso+webkit@gmail.com>
+
+        Web Inspector: Fix spacing of FindBanner prev/next buttons
+        https://bugs.webkit.org/show_bug.cgi?id=149717
+
+        Reviewed by Darin Adler.
+
+        The sizing and spacing of the previous and next buttons in the FindBanner
+        is slightly off and should be readjusted to ensure that both buttons are
+        the same size and have the same spacing.
+
+        * UserInterface/Views/FindBanner.css:
+        (.find-banner > button.segmented.right > .glyph):
+        (.find-banner > button.segmented):
+        (.find-banner > button.segmented > .glyph):
+        (.find-banner > button.segmented.left): Deleted.
+        (.find-banner > button.segmented.left > .glyph): Deleted.
+        (.find-banner > button.segmented.right): Deleted.
+
 2015-10-06  Joseph Pecoraro  <pecoraro@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/FindBanner.css

@@ -144 +144 @@
     border-top-right-radius: 0;
     border-bottom-right-radius: 0;
-
     margin-right: 0;
-
-    padding-left: 6px;
-    padding-right: 6px;
 }
 
 .find-banner > button.segmented.left > .glyph {
     background-image: -webkit-canvas(find-banner-previous-arrow-normal);
-    background-size: 7px 11px;
-    background-repeat: no-repeat;
 }
 
@@ -164 +158 @@
     border-top-left-radius: 0;
     border-bottom-left-radius: 0;
-
     margin-left: -1px;
-
-    padding-left: 5px;
-    padding-right: 7px;
 }
 
 .find-banner > button.segmented.right > .glyph {
     background-image: -webkit-canvas(find-banner-next-arrow-normal);
-    background-size: 7px 11px;
-    background-repeat: no-repeat;
+    margin-left: 1px;
 }
 
@@ -183 +172 @@
 .find-banner > button.segmented {
     min-width: 22px;
-    padding-top: 3px;
-    padding-bottom: 2px;
+    padding: 3px 6px 2px;
+}
+
+.find-banner > button.segmented > .glyph {
+    background-size: 7px 11px;
+    background-repeat: no-repeat;
 }