fix (experiments) #2282 hide Edit Top Sites behind newTabPrefs experi…

…ment
mozilla · Mar 22, 2017 · e693da5 · e693da5
1 parent 5de289f
commit e693da5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/content-src/components/NewTabPage/NewTabPage.js b/content-src/components/NewTabPage/NewTabPage.js
@@ -72,7 +72,8 @@ const NewTabPage = React.createClass({
             <section>
               <TopSites placeholder={!this.props.isReady} page={PAGE_NAME}
                 sites={props.TopSites.rows} showNewStyle={topSitesExperimentIsOn}
-                length={showMoreTopSites ? TOP_SITES_SHOWMORE_LENGTH : TOP_SITES_DEFAULT_LENGTH} />
+                length={showMoreTopSites ? TOP_SITES_SHOWMORE_LENGTH : TOP_SITES_DEFAULT_LENGTH}
+                allowEdit={newTabPrefsExperimentIsOn} />
             </section>
           }
           {showHighlights &&

diff --git a/content-src/components/TopSites/TopSites.js b/content-src/components/TopSites/TopSites.js
@@ -119,7 +119,8 @@ const TopSites = React.createClass({
     return {
       length: TOP_SITES_DEFAULT_LENGTH,
       // This is for event reporting
-      page: "NEW_TAB"
+      page: "NEW_TAB",
+      allowEdit: true
     };
   },
   onClickFactory(index, site) {
@@ -157,7 +158,7 @@ const TopSites = React.createClass({
           );
         })}
       </div>
-      {!this.props.placeholder &&
+      {!this.props.placeholder && this.props.allowEdit &&
         <EditTopSitesIntl {...this.props} />
       }
     </section>);
@@ -181,7 +182,8 @@ TopSites.propTypes = {
    */
   placeholder: React.PropTypes.bool,
 
-  showNewStyle: React.PropTypes.bool
+  showNewStyle: React.PropTypes.bool,
+  allowEdit: React.PropTypes.bool
 };
 
 const EditTopSites = React.createClass({