Bug 1395065 - Move the notification to the top-center of the window r?gasolin draft
authorRicky Chien <ricky060709@gmail.com>
Thu, 07 Sep 2017 18:23:27 +0800
changeset 661371 a2c6d9f196d80895f57fe3dd6fe6b04a727d58f6
parent 661103 b4c1ad9565ee9d00d96501c4a83083daf25c1413
child 730540 cc7a1b3752fa73a98c1c33b0466a6c6a8d578677
push id78725
push userbmo:rchien@mozilla.com
push dateFri, 08 Sep 2017 09:57:22 +0000
reviewersgasolin
bugs1395065
milestone57.0a1
Bug 1395065 - Move the notification to the top-center of the window r?gasolin MozReview-Commit-ID: L4OSbt3hJ4q
browser/base/content/newtab/newTab.css
browser/extensions/onboarding/content/onboarding.css
browser/extensions/onboarding/content/onboarding.js
--- a/browser/base/content/newtab/newTab.css
+++ b/browser/base/content/newtab/newTab.css
@@ -30,16 +30,18 @@ input[type=button] {
   cursor: pointer;
 }
 
 /* UNDO */
 #newtab-undo-container {
   transition: opacity 100ms ease-out;
   -moz-box-align: center;
   -moz-box-pack: center;
+  position: relative;
+  left: -50%;
 }
 
 #newtab-undo-container[undo-disabled] {
   opacity: 0;
   pointer-events: none;
 }
 
 /* CUSTOMIZE */
@@ -60,17 +62,17 @@ input[type=button] {
   display: -moz-box;
   position: relative;
   -moz-box-flex: 1;
   -moz-box-orient: vertical;
 }
 
 #newtab-margin-undo-container {
   display: -moz-box;
-  left: 6px;
+  left: 50%;
   position: absolute;
   top: 6px;
   z-index: 1;
 }
 
 #newtab-margin-undo-container:dir(rtl) {
   left: auto;
   right: 6px;
--- a/browser/extensions/onboarding/content/onboarding.css
+++ b/browser/extensions/onboarding/content/onboarding.css
@@ -19,21 +19,21 @@
   display: none;
 }
 
 #onboarding-overlay.onboarding-opened {
   display: block;
 }
 
 #onboarding-overlay-button {
-  padding: 0;
+  padding: 10px 0 0 0;
   position: absolute;
   cursor: pointer;
-  top: 34px;
-  offset-inline-start: 34px;
+  top: 4px;
+  offset-inline-start: 12px;
   border: none;
   /* Set to none so no grey contrast background in the high-contrast mode */
   background: none;
   /* make sure the icon stay above the activity-stream searchbar */
   /* We want this always under #onboarding-overlay */
   z-index: 10;
 }
 
--- a/browser/extensions/onboarding/content/onboarding.js
+++ b/browser/extensions/onboarding/content/onboarding.js
@@ -17,17 +17,17 @@ const BUNDLE_URI = "chrome://onboarding/
 const UITOUR_JS_URI = "resource://onboarding/lib/UITour-lib.js";
 const TOUR_AGENT_JS_URI = "resource://onboarding/onboarding-tour-agent.js";
 const BRAND_SHORT_NAME = Services.strings
                      .createBundle("chrome://branding/locale/brand.properties")
                      .GetStringFromName("brandShortName");
 const PROMPT_COUNT_PREF = "browser.onboarding.notification.prompt-count";
 const ONBOARDING_DIALOG_ID = "onboarding-overlay-dialog";
 const ONBOARDING_MIN_WIDTH_PX = 960;
-const SPEECH_BUBBLE_MIN_WIDTH_PX = 1150;
+const SPEECH_BUBBLE_MIN_WIDTH_PX = 1130;
 
 /**
  * Add any number of tours, key is the tourId, value should follow the format below
  * "tourId": { // The short tour id which could be saved in pref
  *   // The unique tour id
  *   id: "onboarding-tour-addons",
  *   // The string id of tour name which would be displayed on the navigation bar
  *   tourNameId: "onboarding.tour-addon",