optimized repo-item code

frontend/src/components/repo-view/share-repo-view.js

@@ -73,7 +73,7 @@ class ShareRepoView extends React.Component {
               {(!this.props.isLoading && this.state.repoList.length === 0) && emptyTip}
               {(!this.props.isLoading && this.state.repoList.length > 0) &&
                 <ShareRepoListView 
-                  repoList={this.props.repoList} 
+                  repoList={this.state.repoList} 
                   currentGroup={this.props.currentGroup} 
                   isShowRepoOwner={this.props.isShowRepoOwner}
                 />

frontend/src/components/shared-repo-list-view/shared-repo-list-item.js

@@ -10,6 +10,7 @@ const propTypes = {
   repo: PropTypes.object.isRequired,
   isItemFreezed: PropTypes.bool.isRequired,
   isShowRepoOwner: PropTypes.bool.isRequired,
+  onFreezedItem: PropTypes.func.isRequired,
 };
 
 class RepoListItem extends React.Component {
@@ -21,26 +22,45 @@ class RepoListItem extends React.Component {
       isOperationShow: false,
       isItemMenuShow: false,
     };
+    this.isDeparementOnwerGroupMember = false;
   }
 
   onMouseEnter = () => {
-    this.setState({
+    if (!this.props.isItemFreezed) {
-      highlight: true,
+      this.setState({
-      isOperationShow: true,
+        highlight: true,
-    });
+        isOperationShow: true,
+      });
+    }
   }
   
   onMouseLeave = () => {
-    this.setState({
+    if (!this.props.isItemFreezed) {
-      highlight: false,
+      this.setState({
-      isOperationShow: false,
+        highlight: false,
-    });
+        isOperationShow: false,
+      });
+    }
+  }
+
+  clickOperationMenuToggle = (e) => {
+    e.preventDefault();
+    this.toggleOperationMenu();
   }
 
   toggleOperationMenu = () => {
-    this.setState({
+    if (this.props.isItemFreezed) {
-      isItemMenuShow: !this.state.isItemMenuShow
+      this.setState({
-    });
+        highlight: false,
+        isOperationShow: false,
+        isItemMenuShow: !this.state.isItemMenuShow
+      });
+    } else {
+      this.setState({
+        isItemMenuShow: !this.state.isItemMenuShow
+      });
+    }
+    this.props.onFreezedItem();
   }
 
   getRepoComputeParams = () => {
@@ -67,132 +87,179 @@ class RepoListItem extends React.Component {
     return { iconUrl, iconTitle, libPath };
   }
 
+  onMenuItemClick = (e) => {
+    let operation = e.target.dataset.toggle;
+    switch(operation) {
+      case 'Rename':
+        this.onItemRename();
+        break;
+      case 'Folder Permission':
+        this.onItemPermisionChanged();
+        break;
+      case 'Details':
+        this.onItemDetails();
+        break;
+      case 'Share':
+        this.onItemShared();
+        break;
+      case 'Unshare':
+        this.onItemUnshared();
+        break;
+      default:
+        break;
+    }
+  }
+
+  onItemRename = () => {
+    // todo
+  }
+
+  onItemPermisionChanged = () => {
+    // todo
+  }
+
+  onItemDetails = () => {
+    // todo
+  }
+
+  onItemShared = () => {
+    // todo
+  }
+
+  onItemUnshared = () => {
+    // todo
+  }
+
+  onItemDelete = () => {
+    // todo
+  }
+
   generatorOperations = () => {
     let { repo, currentGroup } = this.props;
     let isStaff = currentGroup.admins.indexOf(username) > -1; //for group repolist;
     let isRepoOwner = repo.owner_email === username;
     let isAdmin = repo.is_admin;
-
+    let operations = [];
-    let iconVisibility = this.state.isOperationShow ? '' : ' invisible';
+    // todo ,shared width me shared width all;
-    let shareIconClassName = 'sf2-icon-share sf2-x repo-share-btn op-icon' + iconVisibility; 
-    let unshareIconClassName = 'sf2-icon-x3 sf2-x op-icon' + iconVisibility; 
-    let deleteIconClassName = 'sf2-icon-delete sf2-x op-icon' + iconVisibility;
-    let operationMenuToggleIconClassName = 'sf2-icon-caret-down item-operation-menu-toggle-icon op-icon';
-    if (window.innerWidth >= 768) {
-      operationMenuToggleIconClassName += iconVisibility;
-    }
-
-    const commonToggle = (
-      <DropdownToggle
-        tag="a" 
-        href="#" 
-        className={operationMenuToggleIconClassName} 
-        title={gettext('More Operations')}
-        onClick={this.clickOperationMenuToggle}
-        data-toggle="dropdown" 
-        aria-expanded={this.state.isItemMenuShow}
-      >
-      </DropdownToggle>
-    );
-
-    const commonOperationsInMenu = (
-      <React.Fragment>
-        <DropdownItem onClick={this.rename}>{gettext('Rename')}</DropdownItem>
-        {folderPermEnabled ? <DropdownItem onClick={this.folderPerm}>{gettext('Folder Permission')}</DropdownItem> : null}
-        <DropdownItem onClick={this.showDetails}>{gettext('Details')}</DropdownItem>
-      </React.Fragment>
-    );
-
-    let desktopOperations;
-    let mobileOperationMenu;
-
-    const share = <a href="#" className={shareIconClassName} title={gettext("Share")} onClick={this.share}></a>;
-    const unshare = <a href="#" className={unshareIconClassName} title={gettext("Unshare")} onClick={this.unshare}></a>
-    const deleteOperation = <a href="#" className={deleteIconClassName} title={gettext('Delete')} onClick={this.deleteItem}></a>;
-
-    const shareDropdownItem = <DropdownItem onClick={this.share}>{gettext('Share')}</DropdownItem>;
-    const unshareDropdownItem = <DropdownItem onClick={this.unshare}>{gettext('Unshare')}</DropdownItem>;
     if (isPro) {
-      if (repo.owner_email.indexOf('@seafile_group') != -1) { // group owned repo
+      if (repo.owner_email.indexOf('@seafile_group') != -1) {  //current repo is belong to a group;
-        if (isStaff) {
+        if (isStaff && repo.owner_email == currentGroup.id + '@seafile_group') { //is a member of this current group,
-          if (repo.owner_email == currentGroup.id + '@seafile_group') { // this repo belongs to the current group
+          this.isDeparementOnwerGroupMember = true;
-            desktopOperations = (
+          if (folderPermEnabled) {
-              <Fragment>
+            operations = ['Rename', 'Folder Permission', 'deatils'];
-                {share}
-                {deleteOperation}
-                <Dropdown isOpen={this.state.isOperationShow} toggle={this.toggleOperationMenu}>
-                  {commonToggle}
-                  <DropdownMenu>
-                    {commonOperationsInMenu}
-                  </DropdownMenu>
-                </Dropdown>
-              </Fragment>
-            );
-            mobileOperationMenu = (
-              <Fragment>
-                {shareDropdownItem}
-                <DropdownItem onClick={this.deleteItem}>{gettext('Delete')}</DropdownItem>
-                {commonOperationsInMenu}
-              </Fragment>
-            );
           } else {
-            desktopOperations = unshare;
+            operations = ['Rename', 'Details']
-            mobileOperationMenu = unshareDropdownItem;
           }
+        } else {
+          operations.push('Unshare');
         }
       } else {
-        desktopOperations = (
+        if (isRepoOwner || isAdmin) {
-          <Fragment>
+          operations.push('Share');
-            {isRepoOwner || isAdmin ? share : null}
+        }
-            {isStaff || isRepoOwner || isAdmin ? unshare : null}
+        if (isStaff || isRepoOwner || isAdmin) {
-          </Fragment>
+          operations.push('Unshare');
-        );
+        }
-        mobileOperationMenu = (
-          <Fragment>
-            {isRepoOwner || isAdmin ? shareDropdownItem : null}
-            {isStaff || isRepoOwner || isAdmin ? unshareDropdownItem : null}
-          </Fragment>
-        );
       }
     } else {
-      desktopOperations = (
+      if (isRepoOwner) {
-        <Fragment>
+        operations.push('share');
-          {isRepoOwner ? share : null}
+      }
-          {isStaff || isRepoOwner ? unshare : null}
+      if (isStaff || isRepoOwner) {
-        </Fragment>
+        operations.push('Unshare');
-      );
+      }
-      mobileOperationMenu = (
-        <Fragment>
-          {isRepoOwner ? shareDropdownItem : null}
-          {isStaff || isRepoOwner ? unshareDropdownItem : null}
-        </Fragment>
-      );
     }
+    return operations;
+  } 
 
-    const mobileOperations = (
+  generatorMobileMenu = () => {
+    let operations = this.generatorOperations();
+    if (this.isDeparementOnwerGroupMember) {
+      operations.unshift('unshare');
+      operations.unshift('share');
+    }
+    return (
       <Dropdown isOpen={this.state.isItemMenuShow} toggle={this.toggleOperationMenu}>
-        {commonToggle}
+        <DropdownToggle 
+          tag="a" 
+          className="sf2-icon-caret-down item-operation-menu-toggle-icon op-icon" 
+          title={gettext('More Operations')} 
+          data-toggle="dropdown" 
+          aria-expanded={this.state.isItemMenuShow}
+          onClick={this.clickOperationMenuToggle}
+        />
         <div className={`${this.state.isItemMenuShow?'':'d-none'}`} onClick={this.toggleOperationMenu}>
           <div className="mobile-operation-menu-bg-layer"></div>
           <div className="mobile-operation-menu">
-            {mobileOperationMenu}
+            {operations.map((item, index) => {
+              return (
+                <DropdownItem key={index} data-toggle={item} onClick={this.onMenuItemClick}>{gettext(item)}</DropdownItem>
+              );
+            })}
           </div>
         </div>
       </Dropdown>
     );
+  }
 
-    return { desktopOperations, mobileOperations }
+  generatorPCMenu = () => {
+    // scene one: (share, delete, itemToggle and other operations);
+    // scene two: (share, unshare), (share), (unshare)
+    let operations = this.generatorOperations();
+    const shareOperation   = <a href="#" className="sf2-icon-share sf2-x op-icon" title={gettext("Share")} onClick={this.onItemShared}></a>;
+    const unshareOperation = <a href="#" className="sf2-icon-x3 sf2-x op-icon" title={gettext("Unshare")} onClick={this.onItemShared}></a>
+    const deleteOperation  = <a href="#" className="sf2-icon-delete sf2-x op-icon" title={gettext('Delete')} onClick={this.onItemDelete}></a>;
+    
+    if (this.isDeparementOnwerGroupMember) {
+      return (
+        <Fragment>
+          {shareOperation}
+          {deleteOperation}
+          <Dropdown isOpen={this.state.isItemMenuShow} toggle={this.toggleOperationMenu}>
+            <DropdownToggle 
+              tag="a" 
+              className="sf2-icon-caret-down item-operation-menu-toggle-icon op-icon" 
+              title={gettext('More Operations')}
+              data-toggle="dropdown" 
+              aria-expanded={this.state.isItemMenuShow}
+              onClick={this.clickOperationMenuToggle}
+            />
+            <DropdownMenu>
+              {operations.map((item, index) => {
+                return <DropdownItem key={index} data-toggle={item} onClick={this.onMenuItemClick}>{gettext(item)}</DropdownItem>
+              })}
+            </DropdownMenu>
+          </Dropdown>
+        </Fragment>
+      );
+    } else {
+      if (operations.length == 2) {
+        return (
+          <Fragment>
+            {shareOperation}
+            {unshareOperation}
+          </Fragment>
+        );
+      }
+      if (operations.length == 1 && operations[0] === 'share') {
+        return shareOperation;
+      }
+
+      if (operations.length == 1 && operations[0] === 'unshare') {
+        return unshareOperation;
+      }
+    }
+    return null;
   }
 
   renderPCUI = () => {
     let { iconUrl, iconTitle, libPath } = this.getRepoComputeParams();
     let { repo, isShowRepoOwner } = this.props;
-    let { desktopOperations } = this.generatorOperations();
     return (
       <tr className={this.state.highlight ? 'tr-highlight' : ''} onMouseEnter={this.onMouseEnter} onMouseLeave={this.onMouseLeave}>
         <td><img src={iconUrl} title={repo.iconTitle} alt={iconTitle} width="24" /></td>
         <td><a href={libPath}>{repo.repo_name}</a></td>
-        <td>{desktopOperations}</td>
+        <td>{this.state.isOperationShow && this.generatorPCMenu()}</td>
         <td>{repo.size}</td>
         <td title={moment(repo.last_modified).format('llll')}>{moment(repo.last_modified).fromNow()}</td>
         {isShowRepoOwner && <td title={repo.owner_contact_email}>{repo.owner_name}</td>}
@@ -203,7 +270,6 @@ class RepoListItem extends React.Component {
   renderMobileUI = () => {
     let { iconUrl, iconTitle, libPath } = this.getRepoComputeParams();
     let { repo, isShowRepoOwner } = this.props;
-    let { mobileOperations } = this.generatorOperations();
     return (
       <tr className={this.state.highlight ? 'tr-highlight' : ''}  onMouseEnter={this.onMouseEnter} onMouseLeave={this.onMouseLeave}>
         <td><img src={iconUrl} title={iconTitle} alt={iconTitle}/></td>
@@ -213,7 +279,7 @@ class RepoListItem extends React.Component {
           <span className="item-meta-info">{repo.size}</span>
           <span className="item-meta-info" title={moment(repo.last_modified).format('llll')}>{moment(repo.last_modified).fromNow()}</span>
         </td>
-        <td>{mobileOperations}</td>
+        <td>{this.generatorMobileMenu()}</td>
       </tr>
     );
   }

frontend/src/components/shared-repo-list-view/shared-repo-list-view.js

@@ -18,6 +18,31 @@ class SharedRepoListView extends React.Component {
     };
   }
 
+  onFreezedItem = () => {
+    this.setState({
+      isItemFreezed: !this.state.isItemFreezed,
+    });
+  }
+
+  renderRepoListView = () => {
+    return (
+      <Fragment>
+        {this.props.repoList.map(repo => {
+          return (
+            <SharedRepoListItem
+              key={repo.repo_id}
+              repo={repo}
+              isShowRepoOwner={this.props.isShowRepoOwner}
+              currentGroup={this.props.currentGroup}
+              isItemFreezed={this.state.isItemFreezed}
+              onFreezedItem={this.onFreezedItem}
+            />
+          );
+        })}
+      </Fragment>
+    );
+  }
+
   renderPCUI = () => {
     let isShowRepoOwner = this.props.isShowRepoOwner;
     return (
@@ -37,17 +62,7 @@ class SharedRepoListView extends React.Component {
           </tr>
         </thead>
         <tbody>
-          {this.props.repoList.map(repo => {
+          {this.renderRepoListView()}
-            return (
-              <SharedRepoListItem
-                key={repo.repo_id}
-                repo={repo}
-                isShowRepoOwner={this.props.isShowRepoOwner}
-                isItemFreezed={this.state.isItemFreezed}
-                currentGroup={this.props.currentGroup}
-              />
-            );
-          })}
         </tbody>
       </table>
     );
@@ -75,17 +90,7 @@ class SharedRepoListView extends React.Component {
           </tr>
         </thead>
         <tbody>
-          {this.props.repoList.map(repo => {
+          {this.renderRepoListView()}
-            return (
-              <SharedRepoListItem
-                key={repo.repo_id}
-                repo={repo}
-                isShowRepoOwner={this.props.isShowRepoOwner}
-                isItemFreezed={this.state.isItemFreezed}
-                currentGroup={this.props.currentGroup}
-              />
-            );
-          })}
         </tbody>
       </table>
     );