Merge pull request #1974 from metabase/fix-pulse-link-macapp

update a couple links that target _blank so that they don't do that from within the Mac app.

frontend/src/admin/datamodel/components/PartialQueryBuilder.jsx

@@ -74,7 +74,7 @@ export default class PartialQueryBuilder extends Component {
                 >
                     <div className="flex align-center mx2 my2">
                         <span className="text-bold px3">{previewSummary}</span>
-                        <a data-metabase-event={"Data Model;Preview Click"} target="_blank" className={cx("Button Button--primary")} href={previewUrl}>Preview</a>
+                        <a data-metabase-event={"Data Model;Preview Click"} target={window.OSX ? null : "_blank"} className={cx("Button Button--primary")} href={previewUrl}>Preview</a>
                     </div>
                 </GuiQueryEditor>
             </div>

frontend/src/pulse/components/SetupMessage.jsx

@@ -30,7 +30,7 @@ export default class SetupMessage extends Component {
                     </div>
                     <div className="mt2">
                         {channels.map(c =>
-                            <a className="Button Button--primary mr1" href={"/admin/settings?section="+c} target="_blank">Configure {c}</a>
+                            <a className="Button Button--primary mr1" href={"/admin/settings?section="+c} target={window.OSX ? null : "_blank"}>Configure {c}</a>
                         )}
                     </div>
                 </div>