diff --git a/packages/client/src/pages/Wizard/index.js b/packages/client/src/pages/Wizard/index.js index 651745b1bf..73f4b700f6 100644 --- a/packages/client/src/pages/Wizard/index.js +++ b/packages/client/src/pages/Wizard/index.js @@ -67,7 +67,6 @@ const Wizard = (props) => { hashSettings, setPortalOwner, setWizardComplete, - getPortalSettings, isLicenseRequired, setLicense, licenseUpload, @@ -253,7 +252,6 @@ const Wizard = (props) => { analytics ); setWizardComplete(); - getPortalSettings(); history.push(combineUrl(window.DocSpaceConfig?.proxy?.url, "/")); } catch (error) { console.error(error); @@ -488,7 +486,6 @@ export default inject(({ auth, wizard }) => { timezone, urlLicense, hashSettings, - getPortalSettings, setWizardComplete, getPortalTimezones, getPortalPasswordSettings, @@ -522,7 +519,6 @@ export default inject(({ auth, wizard }) => { machineName, isLicenseRequired, licenseUpload, - getPortalSettings, setWizardComplete, getPortalPasswordSettings, getPortalTimezones, diff --git a/packages/common/components/Article/index.js b/packages/common/components/Article/index.js index 866337e136..f6bc1fad40 100644 --- a/packages/common/components/Article/index.js +++ b/packages/common/components/Article/index.js @@ -47,8 +47,10 @@ const Article = ({ ...rest }) => { const [articleHeaderContent, setArticleHeaderContent] = React.useState(null); - const [articleMainButtonContent, setArticleMainButtonContent] = - React.useState(null); + const [ + articleMainButtonContent, + setArticleMainButtonContent, + ] = React.useState(null); const [articleBodyContent, setArticleBodyContent] = React.useState(null); const [correctTabletHeight, setCorrectTabletHeight] = React.useState(null); @@ -90,6 +92,8 @@ const Article = ({ }, [children]); const sizeChangeHandler = React.useCallback(() => { + const showArticle = JSON.parse(localStorage.getItem("showArticle")); + if (isMobileOnly || isMobileUtils() || window.innerWidth === 375) { setShowText(true); setIsMobileArticle(true); @@ -98,8 +102,11 @@ const Article = ({ ((isTabletUtils() && window.innerWidth !== 375) || isMobile) && !isMobileOnly ) { - setShowText(false); setIsMobileArticle(true); + + if (showArticle) return; + + setShowText(false); } if (isDesktopUtils() && !isMobile) { setShowText(true); diff --git a/packages/common/components/Article/styled-article.js b/packages/common/components/Article/styled-article.js index 0a12cd204d..500bc368b0 100644 --- a/packages/common/components/Article/styled-article.js +++ b/packages/common/components/Article/styled-article.js @@ -118,6 +118,7 @@ const StyledArticle = styled.article` overflow-x: hidden !important; padding: 0 20px !important; + margin-bottom: 0px !important; @media ${tablet} { padding: 0 8px !important; diff --git a/packages/common/components/Article/sub-components/article-header.js b/packages/common/components/Article/sub-components/article-header.js index a09f079273..755d658897 100644 --- a/packages/common/components/Article/sub-components/article-header.js +++ b/packages/common/components/Article/sub-components/article-header.js @@ -35,29 +35,35 @@ const ArticleHeader = ({ : getLogoFromPath(whiteLabelLogoUrls[0].path.light); if (isMobileOnly) return <>; - return ( - - {isTabletView && isBurgerLoading ? ( - - ) : ( + + const isLoadingComponent = isTabletView ? ( + + ) : ( + + ); + + const mainComponent = ( + <> + {isTabletView && ( )} + + {isTabletView ? ( + + ) : ( + + + + )} + + + ); - {!isTabletView && isBurgerLoading ? ( - - ) : ( - - {isTabletView ? ( - - ) : ( - - - - )} - - )} + return ( + + {isBurgerLoading ? isLoadingComponent : mainComponent} ); }; diff --git a/packages/common/components/Loaders/ArticleFolderLoader/ArticleFolderLoader.js b/packages/common/components/Loaders/ArticleFolderLoader/ArticleFolderLoader.js index 5fec6e6862..161aa76afc 100644 --- a/packages/common/components/Loaders/ArticleFolderLoader/ArticleFolderLoader.js +++ b/packages/common/components/Loaders/ArticleFolderLoader/ArticleFolderLoader.js @@ -12,7 +12,6 @@ const ArticleFolderLoader = ({ className, style, showText, - isVisitor, ...rest }) => { @@ -25,27 +24,54 @@ const ArticleFolderLoader = ({ > {isVisitor ? ( <> - - - + + + - - + + ) : ( <> - - - - + + + + - - - - + + + + )} diff --git a/packages/common/components/Loaders/ArticleFolderLoader/StyledArticleFolderLoader.js b/packages/common/components/Loaders/ArticleFolderLoader/StyledArticleFolderLoader.js index 170e678f7a..821e78fd2a 100644 --- a/packages/common/components/Loaders/ArticleFolderLoader/StyledArticleFolderLoader.js +++ b/packages/common/components/Loaders/ArticleFolderLoader/StyledArticleFolderLoader.js @@ -51,6 +51,17 @@ const StyledBlock = styled.div` @media ${tablet} { margin-bottom: 24px; } + + .article-folder-loader { + @media ${tablet} { + ${(props) => (props.showText ? "width: 200px" : "width: 20px")}; + } + + ${isMobile && + css` + ${(props) => (props.showText ? "width: 200px" : "width: 20px")}; + `} + } `; const StyledRectangleLoader = styled(RectangleLoader)` @@ -60,14 +71,12 @@ const StyledRectangleLoader = styled(RectangleLoader)` @media ${tablet} { height: 20px; - width: 20px; padding: 0 0 24px; } ${isMobile && css` height: 20px; - width: 20px; padding: 0 0 24px; `} diff --git a/packages/common/components/Loaders/ArticleHeaderLoader/ArticleHeaderLoader.js b/packages/common/components/Loaders/ArticleHeaderLoader/ArticleHeaderLoader.js index b7946834c1..a5f772ad7d 100644 --- a/packages/common/components/Loaders/ArticleHeaderLoader/ArticleHeaderLoader.js +++ b/packages/common/components/Loaders/ArticleHeaderLoader/ArticleHeaderLoader.js @@ -1,9 +1,11 @@ import React from "react"; import PropTypes from "prop-types"; +import { inject, observer } from "mobx-react"; + import StyledContainer from "./StyledArticleHeader"; import RectangleLoader from "../RectangleLoader"; -const ArticleHeaderLoader = ({ id, className, style, ...rest }) => { +const ArticleHeaderLoader = ({ id, className, style, showText, ...rest }) => { const { title, width, @@ -17,7 +19,12 @@ const ArticleHeaderLoader = ({ id, className, style, ...rest }) => { animate, } = rest; return ( - + { + return { + showText: auth.settingsStore.showText, + }; +})(observer(ArticleHeaderLoader)); diff --git a/packages/common/components/Loaders/ArticleHeaderLoader/StyledArticleHeader.js b/packages/common/components/Loaders/ArticleHeaderLoader/StyledArticleHeader.js index dd34c070b2..52788cb36c 100644 --- a/packages/common/components/Loaders/ArticleHeaderLoader/StyledArticleHeader.js +++ b/packages/common/components/Loaders/ArticleHeaderLoader/StyledArticleHeader.js @@ -5,12 +5,13 @@ const StyledContainer = styled.div` max-width: 211px; margin-left: 1px; - @media ${tablet} { + ${isMobile} { margin-left: 0; } - ${isMobile} { - margin-left: 0; + @media ${tablet} { + + ${(props) => (props.showText ? "margin-left: 10px" : "margin-left:0")}; } `; diff --git a/packages/common/store/SettingsStore.js b/packages/common/store/SettingsStore.js index f96dfcca94..cd4fa7b4e8 100644 --- a/packages/common/store/SettingsStore.js +++ b/packages/common/store/SettingsStore.js @@ -84,7 +84,7 @@ class SettingsStore { isHeaderVisible = false; isTabletView = false; - showText = false; + showText = JSON.parse(localStorage.getItem("showArticle")) ?? false; articleOpen = false; isMobileArticle = false; @@ -392,7 +392,7 @@ class SettingsStore { this.pluginOptions = origSettings.plugins.allow; } - if (origSettings.tenantAlias) { + if (origSettings?.tenantAlias) { this.setTenantAlias(origSettings.tenantAlias); } }; @@ -650,7 +650,11 @@ class SettingsStore { }; toggleShowText = () => { - this.showText = !this.showText; + const reverseValue = !this.showText; + + localStorage.setItem("showArticle", reverseValue); + + this.showText = reverseValue; }; setArticleOpen = (articleOpen) => {