Bug 867121 - Snapped view: Favicons too close to text in awesome screen.r=mbrubeck
authorAllison Naaktgeboren <ally@mozilla.com>
Tue, 07 May 2013 11:50:28 -0700
changeset 131162 a2eebc32a77ca36c2d0d1a6f06642b589b36144d
parent 131161 0a50d67075bef52c75c79fc3800c31219cf67474
child 131163 74568f858c7cd6eedf2cb5336bf3b34b2da7e03b
push id24649
push userryanvm@gmail.com
push dateWed, 08 May 2013 02:10:32 +0000
treeherdermozilla-central@b980d32c366f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmbrubeck
bugs867121
milestone23.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 867121 - Snapped view: Favicons too close to text in awesome screen.r=mbrubeck
browser/metro/base/content/browser.xul
browser/metro/theme/browser.css
browser/metro/theme/platform.css
--- a/browser/metro/base/content/browser.xul
+++ b/browser/metro/base/content/browser.xul
@@ -279,17 +279,17 @@
             </vbox>
             </scrollbox>
           </hbox>
           <!-- snapped view -->
           <vbox id="snapped-start" class="start-page" observes="bcast_windowState">
             <scrollbox id="snapped-scrollbox" orient="vertical" flex="1">
               <vbox id="snapped-topsites">
                 <label class="meta-section-title" value="&startTopSitesHeader.label;"/>
-                <richgrid id="snapped-topsites-grid" rows="8" columns="1" flex="1"/>
+                <richgrid id="snapped-topsites-grid" class="canSnapTiles" rows="8" columns="1" flex="1"/>
               </vbox>
               <label class="meta-section-title" value="&startBookmarksHeader.label;"
                 onclick="PanelUI.show('bookmarks-container');"/>
               <label class="meta-section-title" value="&startHistoryHeader.label;"
                 onclick="PanelUI.show('history-container');" inputProcessing="true"/>
               <label class="meta-section-title" value="&startRemoteTabsHeader.label;"
                 onclick="PanelUI.show('remotetabs-container');" inputProcessing="true"/>
             </scrollbox>
--- a/browser/metro/theme/browser.css
+++ b/browser/metro/theme/browser.css
@@ -866,16 +866,20 @@ setting[type="radio"] > vbox {
   visibility: collapse;
 }
 
 /*tile content should be on same line in snapped view */
 #snapped-topsites-grid > richgriditem > .richgrid-item-content {
   -moz-box-orient: horizontal;
 }
 
+[viewstate="snapped"] .canSnapTiles .richgrid-item-desc {
+ -moz-margin-start: 8px;
+}
+
 /* if snapped, hide the fullscreen awesome screen, if viewstate is anything
  *  other than snapped, hide the snapped awesome screen */
 #start[viewstate="snapped"],
 #snapped-start:not([viewstate="snapped"]) {
   visibility: collapse;
 }
 
 /*Formatting for the limited horizontal space of snapped*/
--- a/browser/metro/theme/platform.css
+++ b/browser/metro/theme/platform.css
@@ -567,17 +567,17 @@ richgriditem[customColor] .richgrid-icon
 /* hide icon if there is an image background */
 .richgrid-icon-container[customImage] {
   visibility: collapse;
 }
 
 .richgrid-item-desc {
   width: @tile_width@;
   font-size: @metro_font_normal@;
-  margin-left: 0px !important;
+  margin-left: 0px;
   padding-left: 0px !important;
 }
 
 .richgrid-item-content[customImage] > .richgrid-item-desc {
   background: hsla(0,2%,98%,.95);
   /*margin-bottom: 0px;
   margin-right: 0px;*/
   margin: 0px;