diff --git a/client/homebrew/pages/basePages/listPage/listPage.jsx b/client/homebrew/pages/basePages/listPage/listPage.jsx index 7df89ae90..0969b9f2a 100644 --- a/client/homebrew/pages/basePages/listPage/listPage.jsx +++ b/client/homebrew/pages/basePages/listPage/listPage.jsx @@ -1,3 +1,4 @@ +/*eslint max-lines: ["warn", {"max": 300, "skipBlankLines": true, "skipComments": true}]*/ require('./listPage.less'); const React = require('react'); const createClass = require('create-react-class'); @@ -6,7 +7,10 @@ const moment = require('moment'); const BrewItem = require('./brewItem/brewItem.jsx'); -const USERPAGE_KEY_PREFIX = 'HOMEBREWERY-LISTPAGE-VISIBILITY'; +const USERPAGE_KEY_PREFIX = 'HOMEBREWERY-LISTPAGE'; + +const DEFAULT_SORT_TYPE = 'alpha'; +const DEFAULT_SORT_DIR = 'asc'; const ListPage = createClass({ displayName : 'ListPage', @@ -30,10 +34,10 @@ const ListPage = createClass({ }); return { - filterString : this.props.query?.filter || '', - sortType : this.props.query?.sort || 'alpha', - sortDir : this.props.query?.dir || 'asc', - query : this.props.query, + filterString : this.props.query?.filter || '', + sortType : this.props.query?.sort || null, + sortDir : this.props.query?.dir || null, + query : this.props.query, brewCollection : brewCollection }; }, @@ -44,12 +48,19 @@ const ListPage = createClass({ // LOAD FROM LOCAL STORAGE if(typeof window !== 'undefined') { + const newSortType = (this.state.sortType ?? (localStorage.getItem(`${USERPAGE_KEY_PREFIX}-SORTTYPE`) || DEFAULT_SORT_TYPE)); + const newSortDir = (this.state.sortDir ?? (localStorage.getItem(`${USERPAGE_KEY_PREFIX}-SORTDIR`) || DEFAULT_SORT_DIR)); + this.updateUrl(this.state.filterString, newSortType, newSortDir); + const brewCollection = this.props.brewCollection.map((brewGroup)=>{ - brewGroup.visible = (localStorage.getItem(`${USERPAGE_KEY_PREFIX}-${brewGroup.class}`) ?? 'true')=='true'; + brewGroup.visible = (localStorage.getItem(`${USERPAGE_KEY_PREFIX}-VISIBILITY-${brewGroup.class}`) ?? 'true')=='true'; return brewGroup; }); + this.setState({ - brewCollection : brewCollection + brewCollection : brewCollection, + sortType : newSortType, + sortDir : newSortDir }); }; }, @@ -60,8 +71,10 @@ const ListPage = createClass({ saveToLocalStorage : function() { this.state.brewCollection.map((brewGroup)=>{ - localStorage.setItem(`${USERPAGE_KEY_PREFIX}-${brewGroup.class}`, `${brewGroup.visible}`); + localStorage.setItem(`${USERPAGE_KEY_PREFIX}-VISIBILITY-${brewGroup.class}`, `${brewGroup.visible}`); }); + localStorage.setItem(`${USERPAGE_KEY_PREFIX}-SORTTYPE`, this.state.sortType); + localStorage.setItem(`${USERPAGE_KEY_PREFIX}-SORTDIR`, this.state.sortDir); }, renderBrews : function(brews){ @@ -183,11 +196,16 @@ const ListPage = createClass({ getSortedBrews : function(brews){ const testString = _.deburr(this.state.filterString).toLowerCase(); - brews = _.filter(brews, (brew)=>{ - return (_.deburr(brew.title).toLowerCase().includes(testString)) || - (_.deburr(brew.description).toLowerCase().includes(testString)); - }); + brews = _.filter(brews, (brew)=>{ + const brewStrings = _.deburr([ + brew.title, + brew.description, + brew.tags].join('\n') + .toLowerCase()); + + return brewStrings.includes(testString); + }); return _.orderBy(brews, (brew)=>{ return this.sortBrewOrder(brew); }, this.state.sortDir); }, diff --git a/client/homebrew/pages/newPage/newPage.jsx b/client/homebrew/pages/newPage/newPage.jsx index 6a2709896..137ddd996 100644 --- a/client/homebrew/pages/newPage/newPage.jsx +++ b/client/homebrew/pages/newPage/newPage.jsx @@ -27,44 +27,32 @@ const NewPage = createClass({ getDefaultProps : function() { return { brew : { - text : '', - style : undefined, - shareId : null, - editId : null, - createdAt : null, - updatedAt : null, - gDrive : false, - + text : '', + style : undefined, title : '', description : '', - tags : '', - published : false, - authors : [], - systems : [], - renderer : 'legacy', + renderer : 'V3', theme : '5ePHB' } }; }, getInitialState : function() { - const brew = this.props.brew; + let brew = this.props.brew; + + if(this.props.brew.shareId) { + brew = { + text : brew.text ?? '', + style : brew.style ?? undefined, + title : brew.title ?? '', + description : brew.description ?? '', + renderer : brew.renderer ?? 'legacy', + theme : brew.theme ?? '5ePHB' + }; + } return { - brew : { - text : brew.text || '', - style : brew.style || undefined, - gDrive : false, - title : brew.title || '', - description : brew.description || '', - tags : brew.tags || '', - published : false, - authors : [], - systems : brew.systems || [], - renderer : brew.renderer || 'V3', - theme : brew.theme - }, - + brew : brew, isSaving : false, saveGoogle : (global.account && global.account.googleId ? true : false), errors : null, @@ -77,29 +65,26 @@ const NewPage = createClass({ const brew = this.state.brew; - if(typeof window !== 'undefined') { //Load from localStorage if in client browser + if(!this.props.brew.shareId && typeof window !== 'undefined') { //Load from localStorage if in client browser const brewStorage = localStorage.getItem(BREWKEY); const styleStorage = localStorage.getItem(STYLEKEY); const metaStorage = JSON.parse(localStorage.getItem(METAKEY)); + brew.text = brewStorage ?? brew.text; + brew.style = styleStorage ?? brew.style; + // brew.title = metaStorage?.title || this.state.brew.title; + // brew.description = metaStorage?.description || this.state.brew.description; + brew.renderer = metaStorage?.renderer ?? brew.renderer; + brew.renderer = metaStorage?.theme ?? brew.theme; - if(!brew.text || !brew.style){ - brew.text = brew.text || (brewStorage ?? ''); - brew.style = brew.style || (styleStorage ?? undefined); - // brew.title = metaStorage?.title || this.state.brew.title; - // brew.description = metaStorage?.description || this.state.brew.description; - brew.renderer = brew.renderer || metaStorage?.renderer; - brew.theme = brew.theme || metaStorage?.theme; - - this.setState({ - brew : brew - }); - } + this.setState({ + brew : brew + }); } localStorage.setItem(BREWKEY, brew.text); localStorage.setItem(STYLEKEY, brew.style); - localStorage.setItem(METAKEY, JSON.stringify({'renderer' : brew.renderer})); + localStorage.setItem(METAKEY, JSON.stringify({ 'renderer': brew.renderer, 'theme': brew.theme })); }, componentWillUnmount : function() { document.removeEventListener('keydown', this.handleControlKeys); diff --git a/client/homebrew/pages/userPage/userPage.jsx b/client/homebrew/pages/userPage/userPage.jsx index 6c3af7907..7341c61ad 100644 --- a/client/homebrew/pages/userPage/userPage.jsx +++ b/client/homebrew/pages/userPage/userPage.jsx @@ -23,7 +23,7 @@ const UserPage = createClass({ }; }, getInitialState : function() { - const usernameWithS = this.props.username + (this.props.username.endsWith('s') ? `'` : `'s`); + const usernameWithS = this.props.username + (this.props.username.endsWith('s') ? `’` : `’s`); const brews = _.groupBy(this.props.brews, (brew)=>{ return (brew.published ? 'published' : 'private');