From 5b11b87fb5a1add66c320a20fa5cd603e697c6df Mon Sep 17 00:00:00 2001 From: Cassie Tarakajian Date: Tue, 20 Oct 2020 15:35:14 -0400 Subject: [PATCH] [#1590] Connect PreviewFrame in FullView --- .../modules/IDE/components/PreviewFrame.jsx | 48 +++++++++++------ client/modules/IDE/pages/FullView.jsx | 51 +------------------ 2 files changed, 34 insertions(+), 65 deletions(-) diff --git a/client/modules/IDE/components/PreviewFrame.jsx b/client/modules/IDE/components/PreviewFrame.jsx index 08249a3124..d1809e6a9c 100644 --- a/client/modules/IDE/components/PreviewFrame.jsx +++ b/client/modules/IDE/components/PreviewFrame.jsx @@ -359,22 +359,38 @@ PreviewFrame.defaultProps = { cmController: {} }; -const mapStateToProps = state => ({ - files: state.files, - htmlFile: getHTMLFile(state.files), - content: - (state.files.find(file => file.isSelectedFile) || - state.files.find(file => file.name === 'sketch.js') || - state.files.find(file => file.name !== 'root')).content, - isPlaying: state.ide.isPlaying, - isAccessibleOutputPlaying: state.ide.isAccessibleOutputPlaying, - previewIsRefreshing: state.ide.previewIsRefreshing, - textOutput: state.preferences.textOutput, - gridOutput: state.preferences.gridOutput, - soundOutput: state.preferences.soundOutput, - language: state.preferences.language, - autorefresh: state.preferences.autorefresh, -}); +function mapStateToProps(state, ownProps) { + if (ownProps.fullView) { + return { + files: state.files, + htmlFile: getHTMLFile(state.files), + isPlaying: true, + isAccessibleOutputPlaying: false, + textOutput: false, + gridOutput: false, + soundOutput: false, + language: state.preferences.language, + autorefresh: false, + previewIsRefreshing: false + }; + } + return { + files: state.files, + htmlFile: getHTMLFile(state.files), + content: + (state.files.find(file => file.isSelectedFile) || + state.files.find(file => file.name === 'sketch.js') || + state.files.find(file => file.name !== 'root')).content, + isPlaying: state.ide.isPlaying, + isAccessibleOutputPlaying: state.ide.isAccessibleOutputPlaying, + previewIsRefreshing: state.ide.previewIsRefreshing, + textOutput: state.preferences.textOutput, + gridOutput: state.preferences.gridOutput, + soundOutput: state.preferences.soundOutput, + language: state.preferences.language, + autorefresh: state.preferences.autorefresh + }; +} const mapDispatchToProps = { diff --git a/client/modules/IDE/pages/FullView.jsx b/client/modules/IDE/pages/FullView.jsx index f8f9e96fd6..d2f4f57a65 100644 --- a/client/modules/IDE/pages/FullView.jsx +++ b/client/modules/IDE/pages/FullView.jsx @@ -5,7 +5,6 @@ import { bindActionCreators } from 'redux'; import { connect } from 'react-redux'; import PreviewFrame from '../components/PreviewFrame'; import PreviewNav from '../../../components/PreviewNav'; -import { getHTMLFile, getJSFiles, getCSSFiles } from '../reducers/files'; import * as ProjectActions from '../actions/project'; class FullView extends React.Component { @@ -13,8 +12,6 @@ class FullView extends React.Component { this.props.getProject(this.props.params.project_id, this.props.params.username); } - ident = () => {} - render() { return (
@@ -27,26 +24,7 @@ class FullView extends React.Component { />
- } fullView - isPlaying - isAccessibleOutputPlaying={false} - textOutput={false} - gridOutput={false} - soundOutput={false} - dispatchConsoleEvent={this.ident} - endSketchRefresh={this.ident} - previewIsRefreshing={false} - setBlobUrl={this.ident} - stopSketch={this.ident} - expandConsole={this.ident} - clearConsole={this.ident} />
@@ -65,37 +43,12 @@ FullView.propTypes = { username: PropTypes.string }) }).isRequired, - htmlFile: PropTypes.shape({ - id: PropTypes.string.isRequired, - content: PropTypes.string.isRequired, - name: PropTypes.string.isRequired - }).isRequired, - jsFiles: PropTypes.arrayOf(PropTypes.shape({ - id: PropTypes.string.isRequired, - content: PropTypes.string.isRequired, - name: PropTypes.string.isRequired - })).isRequired, - cssFiles: PropTypes.arrayOf(PropTypes.shape({ - id: PropTypes.string.isRequired, - content: PropTypes.string.isRequired, - name: PropTypes.string.isRequired - })).isRequired, - getProject: PropTypes.func.isRequired, - files: PropTypes.arrayOf(PropTypes.shape({ - id: PropTypes.string.isRequired, - content: PropTypes.string.isRequired, - name: PropTypes.string.isRequired - })).isRequired, + getProject: PropTypes.func.isRequired }; function mapStateToProps(state) { return { - user: state.user, - htmlFile: getHTMLFile(state.files), - jsFiles: getJSFiles(state.files), - cssFiles: getCSSFiles(state.files), - project: state.project, - files: state.files + project: state.project }; }