Merge branch 'develop' into feature/portal-access-rights

2022-10-06 13:41:14 +03:00 · 2022-10-06 13:41:14 +03:00 · 5cfc7cf8d0
commit 5cfc7cf8d0
parent ae584739e8 8b03298040
24 changed files with 146 additions and 44 deletions
--- a/build/build.sh
+++ b/build/build.sh
@ -1,12 +1,12 @@
 #!/bin/bash

-@echo off
+echo off

 echo "##########################################################"
 echo "#########  Start build and deploy  #######################"
 echo "##########################################################"

-echo.
+echo ""

 rd="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )"
 echo "Run script directory:" $rd
@ -14,8 +14,22 @@ echo "Run script directory:" $rd
 dir=$(builtin cd $rd/../; pwd)
 echo "Root directory:" $dir

+pushd $dir
+
 echo "FRONT-END (for start run command 'yarn start' inside the root folder)"
-yarn install $dir
+yarn install

 echo "BACK-END"
 dotnet build $dir/asc.web.slnf  /fl1 /flp1:logfile=asc.web.log;verbosity=normal
+
+echo "install nodejs projects dependencies..."
+pushd $dir/common/ASC.Socket.IO/
+yarn install
+pushd $dir/common/ASC.SsoAuth/
+yarn install
+pushd $dir/common/ASC.WebDav/
+yarn install
+pushd $dir/common/ASC.UrlShortener/
+yarn install
+pushd $dir/common/ASC.WebPlugins/
+yarn install
--- a/build/run/macos/socket.sh
+++ b/build/run/macos/socket.sh
@ -0,0 +1,10 @@
+#!/bin/bash
+
+rd="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )"
+echo "Run script directory:" $rd
+
+dir=$(builtin cd $rd/../../../; pwd)
+echo "Root directory:" $dir
+
+# Web Socket IO
+node $dir/common/ASC.Socket.IO/server.js --logPath=$dir/Logs 
--- a/build/runMigrations.sh
+++ b/build/runMigrations.sh
--- a/build/start/command.ps1
+++ b/build/start/command.ps1
@ -13,7 +13,7 @@ $CommandName = "$($args[0])";
 #Write-Output  "Starting $($CommandName) services at time: $(Get-Date -Format HH:mm:ss)"
 #Write-Output  ""

-Get-ChildItem -Path $WorkDir | ForEach-Object -ThrottleLimit 20 -Parallel  {
+Get-ChildItem -Path $WorkDir -File | ForEach-Object -ThrottleLimit 20 -Parallel  {
    $ServiceName = "Onlyoffice$([System.IO.Path]::GetFileNameWithoutExtension($_))";

    switch ( $Using:CommandName )
--- a/frontend.macos.code-workspace
+++ b/frontend.macos.code-workspace
@ -0,0 +1,43 @@
+{
+  "folders": [
+    {
+      "name": "🌐 root",
+      "path": "."
+    },
+    {
+      "name": "🚀 @docspace/client",
+      "path": "packages/client"
+    },
+    {
+      "name": "🔑 @docspace/login",
+      "path": "packages/login"
+    },
+    {
+      "name": "📄 @docspace/editor",
+      "path": "packages/editor"
+    },
+    {
+      "name": "📦 @docspace/common",
+      "path": "packages/common"
+    },
+    {
+      "name": "📦 @docspace/components",
+      "path": "packages/components"
+    }
+  ],
+  "settings": {
+    "window.zoomLevel": 0,
+    "editor.formatOnSave": true,
+    "editor.defaultFormatter": "esbenp.prettier-vscode",
+    "liveServer.settings.multiRootWorkspaceName": "✨ docspace",
+    "cSpell.words": ["docspace", "browserslist", "debuginfo", "doceditor"],
+    "jest.autoRun": "false"
+  },
+  "extensions": {
+    "recommendations": [
+      "folke.vscode-monorepo-workspace",
+      "orta.vscode-jest",
+      "firsttris.vscode-jest-runner"
+    ]
+  }
+}
--- a/packages/client/src/components/EmptyContainer/EmptyContainer.js
+++ b/packages/client/src/components/EmptyContainer/EmptyContainer.js
@ -7,6 +7,7 @@ import {
  smallTablet,
  desktop,
 } from "@docspace/components/utils/device";
+import { isMobile } from "react-device-detect";

 const EmptyPageStyles = css`
  padding: 44px 0px 64px 0px;
@ -29,8 +30,9 @@ const EmptyPageStyles = css`
  }

  @media ${tablet} {
-    padding: 44px 0px 64px 97px;
+    padding: 44px 0px 64px 0px;
    grid-column-gap: 33px;
+    margin-left: auto;
  }

  @media ${smallTablet} {
@ -88,6 +90,13 @@ const EmptyFolderWrapper = styled.div`
      bottom: 16px;
    }

+    @media screen and (max-width: 1325px) {
+      ${!isMobile &&
+      css`
+        margin-left: 98px;
+      `};
+    }
+
    ${(props) => props.isEmptyPage && `${EmptyPageStyles}`}
  }
 `;
--- a/packages/client/src/pages/Home/Section/Body/RowsView/FilesRowContainer.js
+++ b/packages/client/src/pages/Home/Section/Body/RowsView/FilesRowContainer.js
@ -50,7 +50,7 @@ const StyledRowContainer = styled(RowContainer)`
    .files-row {
      border-top: ${(props) =>
        `1px ${props.theme.filesSection.tableView.row.borderColor} solid`};
-      margin-top: -3px;
+      margin-top: -1px;
      ${marginStyles}
    }
  }
@ -94,7 +94,7 @@ const FilesRowContainer = ({
      hasMoreFiles={hasMoreFiles}
      draggable
      useReactWindow={!withPaging}
-      itemHeight={58}
+      itemHeight={59}
    >
      {filesList.map((item, index) => (
        <SimpleFilesRow
--- a/packages/client/src/pages/Home/Section/Body/RowsView/SimpleFilesRow.js
+++ b/packages/client/src/pages/Home/Section/Body/RowsView/SimpleFilesRow.js
@ -39,7 +39,7 @@ const StyledWrapper = styled.div`
 const StyledSimpleFilesRow = styled(Row)`
  ${(props) => (props.checked || props.isActive) && checkedStyle};
  ${(props) => props.dragging && draggingStyle}
-  height: 57px;
+  height: 56px;

  ${(props) =>
    !isMobile &&
@ -49,7 +49,7 @@ const StyledSimpleFilesRow = styled(Row)`
        cursor: pointer;
        ${checkedStyle}

-        margin-top: -3px;
+        margin-top: -1px;
        border-top: ${(props) =>
          `1px ${props.theme.filesSection.tableView.row.borderColor} solid`};
      }
@ -67,13 +67,13 @@ const StyledSimpleFilesRow = styled(Row)`
      /* cursor: wait; */
    `}

-  margin-top: -2px;
+  margin-top: 0px;

  ${(props) =>
    props.showHotkeyBorder &&
    css`
      border-top: 1px solid #2da7db !important;
-      margin-top: -3px;
+      margin-top: -1px;
      margin-left: -24px;
      margin-right: -24px;
      padding-left: 24px;
--- a/packages/client/src/pages/PortalSettings/Layout/Article/Body/index.js
+++ b/packages/client/src/pages/PortalSettings/Layout/Article/Body/index.js
@ -210,7 +210,7 @@ class ArticleBodyContent extends React.Component {
          showText={showText}
          text={this.mapKeys(item.tKey)}
          value={item.link}
-          isActive={item.key + "-0" === selectedKeys[0]}
+          isActive={item.key === selectedKeys[0][0]}
          onClick={() => this.onSelect(item.key)}
        />
      );
--- a/packages/client/src/pages/Profile/index.js
+++ b/packages/client/src/pages/Profile/index.js
@ -21,11 +21,13 @@ class Profile extends React.Component {
      setIsLoading,
      setIsEditTargetUser,
      setLoadedProfile,
+      setSelectedNode,
    } = this.props;
    const userId = "@self";

    setFirstLoad(false);
    setIsEditTargetUser(false);
+    setSelectedNode(["accounts"]);

    setDocumentTitle(t("Common:Profile"));
    this.documentElement = document.getElementsByClassName("hidingHeader");
@ -101,7 +103,7 @@ Profile.propTypes = {
 };

 export default withRouter(
-  inject(({ auth, peopleStore }) => {
+  inject(({ auth, peopleStore, treeFoldersStore }) => {
    const { setDocumentTitle, isAdmin, language } = auth;
    const { targetUserStore, loadingStore } = peopleStore;
    const {
@ -111,6 +113,7 @@ export default withRouter(
      setIsEditTargetUser,
    } = targetUserStore;
    const { setFirstLoad, setIsLoading, setLoadedProfile } = loadingStore;
+    const { setSelectedNode } = treeFoldersStore;
    return {
      setDocumentTitle,
      isAdmin,
@ -123,6 +126,7 @@ export default withRouter(
      setIsEditTargetUser,
      setLoadedProfile,
      showCatalog: auth.settingsStore.showCatalog,
+      setSelectedNode,
    };
  })(
    observer(withTranslation(["Profile", "Common"])(withCultureNames(Profile)))
--- a/packages/client/src/store/ContextOptionsStore.js
+++ b/packages/client/src/store/ContextOptionsStore.js
@ -551,7 +551,7 @@ class ContextOptionsStore {
      {
        key: "room-info",
        label: "Info",
-        icon: "/static/images/info.react.svg",
+        icon: "/static/images/info.outline.react.svg",
        onClick: this.onShowInfoPanel,
        disabled: false,
      },
--- a/packages/client/src/store/FilesActionsStore.js
+++ b/packages/client/src/store/FilesActionsStore.js
@ -664,7 +664,7 @@ class FilesActionStore {
          toastr.success(translations.successRemoveFolder);
        } else {
          this.updateFilesAfterDelete([itemId]);
-          this.filesStore.removeFiles([itemId], null, () =>
+          this.filesStore.removeFiles(null, [itemId], () =>
            toastr.success(translations.successRemoveFolder)
          );
        }
--- a/packages/client/src/store/FilesStore.js
+++ b/packages/client/src/store/FilesStore.js
@ -1702,7 +1702,8 @@ class FilesStore {

  removeFiles = (fileIds, folderIds, showToast) => {
    const newFilter = this.filter.clone();
-    const deleteCount = fileIds.length + folderIds.length;
+    const deleteCount = (fileIds?.length ?? 0) + (folderIds?.length ?? 0);
+
    newFilter.startIndex =
      (newFilter.page + 1) * newFilter.pageCount - deleteCount;
    newFilter.pageCount = deleteCount;
@ -1712,10 +1713,10 @@ class FilesStore {
      .then((res) => {
        const files = fileIds
          ? this.files.filter((x) => !fileIds.includes(x.id))
-          : [];
+          : this.files;
        const folders = folderIds
          ? this.folders.filter((x) => !folderIds.includes(x.id))
-          : [];
+          : this.folders;

        const newFiles = [...files, ...res.files];
        const newFolders = [...folders, ...res.folders];
--- a/packages/client/src/store/ProfileActionsStore.js
+++ b/packages/client/src/store/ProfileActionsStore.js
@ -190,7 +190,7 @@ class ProfileActionsStore {
      hotkeys,
      {
        key: "AboutBtn",
-        icon: "/static/images/info.react.svg",
+        icon: "/static/images/info.outline.react.svg",
        label: t("Common:AboutCompanyTitle"),
        onClick: this.onAboutClick,
      },
@ -210,7 +210,7 @@ class ProfileActionsStore {
    if (debugInfo) {
      actions.splice(3, 0, {
        key: "DebugBtn",
-        icon: "/static/images/info.react.svg",
+        icon: "/static/images/info.outline.react.svg",
        label: "Debug Info",
        onClick: this.onDebugClick,
      });
--- a/packages/common/components/Article/styled-article.js
+++ b/packages/common/components/Article/styled-article.js
@ -289,6 +289,10 @@ const StyledArticleProfile = styled.div`
    padding: 16px 14px;
  `}

+  .profile-avatar {
+    cursor: pointer;
+  }
+
  .option-button {
    margin-left: auto;
    height: 32px;
--- a/packages/common/components/Article/sub-components/article-profile.js
+++ b/packages/common/components/Article/sub-components/article-profile.js
@ -15,7 +15,7 @@ import {
 } from "../styled-article";

 const ArticleProfile = (props) => {
-  const { user, showText, getUserRole, getActions } = props;
+  const { user, showText, getUserRole, getActions, onProfileClick } = props;
  const { t } = useTranslation("Common");
  const [isOpen, setIsOpen] = useState(false);
  const ref = useRef(null);
@ -31,7 +31,11 @@ const ArticleProfile = (props) => {
  };

  const onAvatarClick = (e) => {
-    if (isTabletView && !showText) toggle(e, !isOpen);
+    if (isTabletView && !showText) {
+      toggle(e, !isOpen);
+    } else {
+      onProfileClick();
+    }
  };

  const onHide = () => {
@ -46,6 +50,7 @@ const ArticleProfile = (props) => {
      <StyledArticleProfile showText={showText} tablet={isTabletView}>
        <div ref={ref}>
          <Avatar
+            className={"profile-avatar"}
            size={avatarSize}
            role={"user"}
            source={user.avatar}
@ -94,9 +99,10 @@ const ArticleProfile = (props) => {

 export default withRouter(
  inject(({ auth, profileActionsStore }) => {
-    const { getActions, getUserRole } = profileActionsStore;
+    const { getActions, getUserRole, onProfileClick } = profileActionsStore;

    return {
+      onProfileClick,
      user: auth.userStore.user,
      getUserRole,
      getActions,
--- a/packages/components/combobox/styled-combobox.js
+++ b/packages/components/combobox/styled-combobox.js
@ -19,7 +19,7 @@ const StyledComboBox = styled.div`
    props.isOpen &&
    props.noBorder &&
    css`
-      background: #dfe2e3;
+      background: ${(props) => props.theme.comboBox.background};
      border-radius: 3px;
    `}

--- a/packages/components/drop-down-item/styled-drop-down-item.js
+++ b/packages/components/drop-down-item/styled-drop-down-item.js
@ -63,6 +63,13 @@ const StyledDropdownItem = styled.div`
            ? props.theme.dropDownItem.icon.disableColor
            : props.theme.dropDownItem.icon.color};
      }
+
+      rect[fill] {
+        fill: ${(props) =>
+          props.disabled
+            ? props.theme.dropDownItem.icon.disableColor
+            : props.theme.dropDownItem.icon.color};
+      }
    }
  }

--- a/packages/components/empty-screen-container/styled-empty-screen-container.js
+++ b/packages/components/empty-screen-container/styled-empty-screen-container.js
@ -72,6 +72,7 @@ const EmptyContentBody = styled.div`
  grid-column-gap: 16px;
  grid-row-gap: 10px;
  max-width: 800px;
+
  grid-template-rows: max-content;
  .ec-image {
    grid-area: img;
@ -79,6 +80,10 @@ const EmptyContentBody = styled.div`
    ${NoUserSelect}
  }

+  @media ${tablet} {
+    max-width: 480px;
+  }
+
  .ec-header {
    grid-area: headerText;
    padding-top: 16px;
@ -104,7 +109,6 @@ const EmptyContentBody = styled.div`
  @media (orientation: portrait) {
    @media (max-width: 768px) {
      padding-top: 0px;
-      max-width: 700px;

      .ec-image {
        max-height: 100px;
--- a/packages/components/row-content/styled-row-content.js
+++ b/packages/components/row-content/styled-row-content.js
@ -79,7 +79,9 @@ const MainContainerWrapper = styled.div`
      props.widthProp &&
      props.widthProp < size.tablet) ||
    props.isMobile
-      ? `${mainWrapperTabletStyle}`
+      ? css`
+          ${mainWrapperTabletStyle}
+        `
      : `
  `}
  @media ${tablet} {
--- a/packages/components/themes/base.js
+++ b/packages/components/themes/base.js
@ -1528,6 +1528,7 @@ const Base = {

  comboBox: {
    padding: "6px 0px",
+    background: lightGrayishStrongBlue,

    width: {
      base: "173px",
--- a/packages/components/themes/dark.js
+++ b/packages/components/themes/dark.js
@ -1525,6 +1525,7 @@ const Dark = {

  comboBox: {
    padding: "6px 0px",
+    background: black,

    width: {
      base: "173px",
@ -1614,10 +1615,11 @@ const Dark = {
  },

  toggleButton: {
-    fillColor: grayMaxLight,
+    fillColor: "#F0F0F0",
    fillColorOff: "#292929",

    disableFillColor: black,
+    disableFillColorOff: "#545454",

    borderColor: "#474747",
    borderColorOff: "#474747",
@ -1625,11 +1627,11 @@ const Dark = {
    disableBorderColor: "#474747",
    disableBorderColorOff: "#646464",

-    fillCircleColor: "#FFFFFF",
+    fillCircleColor: "#292929",
    fillCircleColorOff: grayMaxLight,

-    disableFillCircleColor: "#333333",
-    disableFillCircleColorOff: "#797979",
+    disableFillCircleColor: "#545454",
+    disableFillCircleColorOff: black,
  },

  contextMenuButton: {
--- a/packages/components/toggle-button/index.js
+++ b/packages/components/toggle-button/index.js
@ -1,15 +1,8 @@
 import React, { Component } from "react";
 import PropTypes from "prop-types";
-import {
-  ToggleButtonContainer,
-  HiddenInput,
-  Container,
-} from "./styled-toggle-button";
+import { ToggleButtonContainer, HiddenInput } from "./styled-toggle-button";
 import Text from "../text";
-import globalColors from "../utils/globalColors";
 import { motion } from "framer-motion";
-import Base from "../themes/base";
-
 import { ColorTheme, ThemeType } from "@docspace/common/components/ColorTheme";

 const ToggleIcon = ({ isChecked, isLoading }) => {
@ -72,16 +65,12 @@ class ToggleButton extends Component {
      className,
      style,
      isLoading,
-      theme,
    } = this.props;
-    const { gray } = globalColors;
-    const colorProps = isDisabled ? { color: gray } : {};

    //console.log("ToggleButton render");

    return (
      <ColorTheme
-        theme={theme}
        themeId={ThemeType.ToggleButton}
        id={id}
        className={className}
@ -102,7 +91,7 @@ class ToggleButton extends Component {
          />
          <ToggleIcon isChecked={this.state.checked} isLoading={isLoading} />
          {label && (
-            <Text className="toggle-button-text" as="span" {...colorProps}>
+            <Text className="toggle-button-text" as="span">
              {label}
            </Text>
          )}
--- a/packages/components/toggle-button/styled-toggle-button.js
+++ b/packages/components/toggle-button/styled-toggle-button.js
@ -36,9 +36,12 @@ const ToggleButtonContainer = styled.label`
            rect {
              fill: ${props.isChecked
                ? props.theme.toggleButton.disableFillColor
-                : props.theme.toggleButton.fillColorOff};
+                : props.theme.toggleButton.disableFillColorOff};
              stroke-width: 1px;
              stroke-linecap: round;
+              stroke: ${props.isChecked
+                ? props.theme.toggleButton.borderColor
+                : props.theme.toggleButton.borderColorOff};
            }
            circle {
              fill: ${props.isChecked
@ -52,6 +55,9 @@ const ToggleButtonContainer = styled.label`
                ? props.theme.toggleButton.fillColor
                : props.theme.toggleButton.fillColorOff};
              stroke-width: 1px;
+              stroke: ${props.isChecked
+                ? props.theme.toggleButton.borderColor
+                : props.theme.toggleButton.borderColor};
            }
            circle {
              fill: ${props.isChecked