Merge pull request scratchfoundation#4710 from benjiwheeler/only-autosave-if-changed

benjiwheeler · web-flow · commit 5c1d3e38bda3 · 2019-04-17T14:55:22.000-04:00
Share, See-project-page buttons will only save if project has changed
diff --git a/src/components/menu-bar/menu-bar.jsx b/src/components/menu-bar/menu-bar.jsx
@@ -194,7 +194,7 @@ class MenuBar extends React.Component {
         this.props.onRequestCloseFile();
     }
     handleClickSeeCommunity (waitForUpdate) {
-        if (this.props.canSave) { // save before transitioning to project page
+        if (this.props.canSave && this.props.projectChanged) { // save before transitioning to project page
             this.props.autoUpdateProject();
             waitForUpdate(true); // queue the transition to project page
         } else {

Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,7 @@ class MenuBar extends React.Component {`
`194`	`194`	`this.props.onRequestCloseFile();`
`195`	`195`	`}`
`196`	`196`	`handleClickSeeCommunity (waitForUpdate) {`
`197`		`- if (this.props.canSave) { // save before transitioning to project page`
	`197`	`+ if (this.props.canSave && this.props.projectChanged) { // save before transitioning to project page`
`198`	`198`	`this.props.autoUpdateProject();`
`199`	`199`	`waitForUpdate(true); // queue the transition to project page`
`200`	`200`	`} else {`