Themes: source prop for thanks modal

Automattic · May 9, 2016 · f6d976e · f6d976e
1 parent e9e231b
commit f6d976e
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 10 deletions.
diff --git a/client/my-sites/theme/main.jsx b/client/my-sites/theme/main.jsx
@@ -227,7 +227,7 @@ const ThemeSheet = React.createClass( {
 				<ActivatingTheme siteId={ siteID }>
 					<ThanksModal
 						site={ this.props.selectedSite }
-						topLink={ 'customize' }
+						source={ 'details' }
 						clearActivated={ bindActionCreators( clearActivated, this.props.dispatch ) }/>
 				</ActivatingTheme>
 				<div className="themes__sheet-columns">

diff --git a/client/my-sites/themes/single-site.jsx b/client/my-sites/themes/single-site.jsx
@@ -150,6 +150,7 @@ var ThemesSingleSite = React.createClass( {
 				<ActivatingTheme siteId={ site.ID } >
 					<ThanksModal
 						site={ site }
+						source={ 'list' }
 						clearActivated={ bindActionCreators( Action.clearActivated, this.props.dispatch ) } />
 				</ActivatingTheme>
 				<CurrentThemeData site={ site }>

diff --git a/client/my-sites/themes/thanks-modal.jsx b/client/my-sites/themes/thanks-modal.jsx
@@ -15,14 +15,8 @@ var ThanksModal = React.createClass( {
 
 	propTypes: {
 		clearActivated: React.PropTypes.func.isRequired,
-		// First link to show for wpcom themes
-		topLink: React.PropTypes.oneOf( [ 'features', 'customize' ] ),
-	},
-
-	getDefaultProps: function() {
-		return {
-			topLink: 'features',
-		};
+		// Where is the modal being used?
+		source: React.PropTypes.oneOf( [ 'details', 'list' ] ).isRequired,
 	},
 
 	onCloseModal: function() {
@@ -54,7 +48,7 @@ var ThanksModal = React.createClass( {
 		return (
 			<ul>
 				<li>
-					{ this.props.topLink === 'features' ? features : customize }
+					{ this.props.source === 'list' ? features : customize }
 				</li>
 			<li>
 				{ this.translate( 'Have questions? Stop by our {{a}}support forums.{{/a}}', {