Bug 1167805 - Not enough padding for page title causing pin icon to overlap [r=adw, a=lizzard]
authorEd Lee <edilee@mozilla.com>
Fri, 22 May 2015 17:37:21 -0700
changeset 273276 06ba020c6f0f7d654c1f3378596525f73415f8ca
parent 273275 8193c7e6c464edb414b37c0f0cc5453d4eb9d54c
child 273277 e5bb6c8253ed3177c8c3307fe8cfb88316363e07
push id4830
push userjlund@mozilla.com
push dateMon, 29 Jun 2015 20:18:48 +0000
treeherdermozilla-beta@4c2175bb0420 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersadw, lizzard
bugs1167805
milestone40.0a2
Bug 1167805 - Not enough padding for page title causing pin icon to overlap [r=adw, a=lizzard]
browser/base/content/newtab/newTab.css
browser/themes/shared/newtab/newTab.inc.css
--- a/browser/base/content/newtab/newTab.css
+++ b/browser/base/content/newtab/newTab.css
@@ -242,16 +242,17 @@ input[type=button] {
 }
 
 .newtab-suggested-bounds {
   max-height: 34px; /* 34 / 17 = 2 lines maximum */
 }
 
 .newtab-title {
   left: 0;
+  padding: 0 4px;
 }
 
 .newtab-sponsored {
   border: 1px solid #dcdcdc;
   border-radius: 2px;
   cursor: pointer;
   display: none;
   font-family: Arial;
--- a/browser/themes/shared/newtab/newTab.inc.css
+++ b/browser/themes/shared/newtab/newTab.inc.css
@@ -168,17 +168,17 @@
   color: white;
 }
 
 .newtab-site:hover .newtab-title {
   color: #222;
 }
 
 .newtab-site[pinned] .newtab-title {
-  padding: 0 15px;
+  -moz-padding-start: 24px;
 }
 
 .newtab-site[pinned] .newtab-title::before {
   background-image: -moz-image-rect(url("chrome://browser/skin/newtab/controls.svg"), 7, 278, 28, 266);
   background-size: 10px;
   content: "";
   height: 17px;
   left: 0;