From dd2a8202ef61b0b83a517b32f3aa1757c93c061e Mon Sep 17 00:00:00 2001 From: Rokt33r Date: Mon, 28 Dec 2015 21:01:43 +0900 Subject: [PATCH] bug fix --- browser/lib/dataStore.js | 4 +++- browser/main/HomePage/ArticleDetail/index.js | 15 ++++++--------- browser/main/HomePage/ArticleNavigator.js | 10 ++++++---- browser/main/HomePage/ArticleTopBar.js | 4 +++- 4 files changed, 18 insertions(+), 15 deletions(-) diff --git a/browser/lib/dataStore.js b/browser/lib/dataStore.js index 48d00a27..60e3577c 100644 --- a/browser/lib/dataStore.js +++ b/browser/lib/dataStore.js @@ -93,7 +93,9 @@ export function init () { content: defaultContent, mode: 'markdown', key: keygen(), - FolderKey: defaultFolder.key + FolderKey: defaultFolder.key, + createdAt: new Date(), + updatedAt: new Date() } data = { diff --git a/browser/main/HomePage/ArticleDetail/index.js b/browser/main/HomePage/ArticleDetail/index.js index ffde0fa8..493fb92b 100644 --- a/browser/main/HomePage/ArticleDetail/index.js +++ b/browser/main/HomePage/ArticleDetail/index.js @@ -20,6 +20,7 @@ import DeleteArticleModal from '../../modal/DeleteArticleModal' const electron = require('electron') const clipboard = electron.clipboard +const OSX = process.platform === 'darwin' const BRAND_COLOR = '#18AF90' const editDeleteTutorialElement = ( @@ -83,16 +84,12 @@ function notify (...args) { return new window.Notification(...args) } -function makeInstantArticle (article) { - return Object.assign({}, article) -} - export default class ArticleDetail extends React.Component { constructor (props) { super(props) this.state = { - article: makeInstantArticle(props.activeArticle), + article: Object.assign({content: ''}, props.activeArticle), previewMode: false, isArticleEdited: false, isTagChanged: false, @@ -121,7 +118,7 @@ export default class ArticleDetail extends React.Component { let nextArticle = nextProps.activeArticle let nextModified = nextArticle != null ? _.findWhere(nextProps.modified, {key: nextArticle.key}) : null - let article = Object.assign({}, nextProps.activeArticle, nextModified) + let article = Object.assign({content: ''}, nextProps.activeArticle, nextModified) let nextState = { article, previewMode: false @@ -304,7 +301,7 @@ export default class ArticleDetail extends React.Component { { this.state.article.mode === 'markdown' ? : null } @@ -319,10 +316,10 @@ export default class ArticleDetail extends React.Component { diff --git a/browser/main/HomePage/ArticleNavigator.js b/browser/main/HomePage/ArticleNavigator.js index 926123f7..1e77d7cb 100644 --- a/browser/main/HomePage/ArticleNavigator.js +++ b/browser/main/HomePage/ArticleNavigator.js @@ -66,7 +66,8 @@ export default class ArticleNavigator extends React.Component { let { dispatch, folders, status } = this.props let { targetFolders } = status - let FolderKey = targetFolders.length > 0 + let isFolderFilterApplied = targetFolders.length > 0 + let FolderKey = isFolderFilterApplied ? targetFolders[0].key : folders[0].key @@ -82,11 +83,12 @@ export default class ArticleNavigator extends React.Component { } dispatch(saveArticle(newArticle.key, newArticle, true)) + if (isFolderFilterApplied) dispatch(switchFolder(targetFolders[0].name)) } handleNewFolderButton (e) { - let { activeUser } = this.props - openModal(CreateNewFolder, {user: activeUser}) + let { user } = this.props + openModal(CreateNewFolder, {user: user}) } handleFolderButtonClick (name) { @@ -163,7 +165,7 @@ export default class ArticleNavigator extends React.Component { } ArticleNavigator.propTypes = { - activeUser: PropTypes.object, + user: PropTypes.object, folders: PropTypes.array, allArticles: PropTypes.array, status: PropTypes.shape({ diff --git a/browser/main/HomePage/ArticleTopBar.js b/browser/main/HomePage/ArticleTopBar.js index d4a9b2c2..f0f2fb21 100644 --- a/browser/main/HomePage/ArticleTopBar.js +++ b/browser/main/HomePage/ArticleTopBar.js @@ -9,6 +9,8 @@ const remote = electron.remote const Menu = remote.Menu const MenuItem = remote.MenuItem +const OSX = process.platform === 'darwin' + var menu = new Menu() var lastIndex = -1 menu.append(new MenuItem({ @@ -191,7 +193,7 @@ export default class ArticleTopBar extends React.Component {