Merge pull request #10104 from BerkeleyTrue/fix/challenge-page-title

Fix(meta): challenge should use challenge title for page title

common/app/routes/challenges/components/Show.jsx

@@ -39,7 +39,8 @@ const mapStateToProps = createSelector(
   challengeSelector,
   state => state.challengesApp.challenge,
   state => state.challengesApp.superBlocks,
-    ({ viewType }, challenge, superBlocks = []) => ({
+  ({ challenge: { title } = {}, viewType }, challenge, superBlocks = []) => ({
+    title,
     challenge,
     viewType,
     areChallengesLoaded: superBlocks.length > 0
@@ -60,6 +61,7 @@ export class Challenges extends PureComponent {
   static displayName = 'Challenges';
 
   static propTypes = {
+    title: PropTypes.string,
     isStep: PropTypes.bool,
     fetchChallenges: PropTypes.func.isRequired,
     replaceChallenge: PropTypes.func.isRequired,
@@ -70,7 +72,7 @@ export class Challenges extends PureComponent {
   };
 
   componentWillMount() {
-    this.props.updateTitle(this.props.params.dashedName);
+    this.props.updateTitle(this.props.title);
   }
 
   componentDidMount() {
@@ -87,7 +89,7 @@ export class Challenges extends PureComponent {
     const { block, dashedName } = nextProps.params;
     const { resetUi, updateTitle, replaceChallenge } = this.props;
     if (this.props.params.dashedName !== dashedName) {
-      updateTitle(dashedName);
+      updateTitle(nextProps.title);
       resetUi();
       replaceChallenge({ dashedName, block });
     }

common/app/routes/challenges/components/map/Map.jsx

@@ -37,6 +37,11 @@ export class ShowMap extends PureComponent {
   };
 
   componentWillMount() {
+    // if no params then map is open in drawer
+    // do not update title
+    if (!this.props.params) {
+      return;
+    }
     this.props.updateTitle(
       'A Map to Learn to Code and Become a Software Engineer'
     );