From c4165798c4a6c67c523a530f587b561ebc34f951 Mon Sep 17 00:00:00 2001
From: shanshuirenjia <978987373@qq.com>
Date: Mon, 22 Apr 2019 15:15:51 +0800
Subject: [PATCH] improve class naming

---
 .../components/dirent-list-view/dirent-list-item.js  |  4 ++--
 .../components/dropdown-menu/item-dropdown-menu.js   | 12 ++++++------
 .../toolbar/mutilple-dir-operation-toolbar.js        |  2 +-
 frontend/src/components/tree-view/tree-node-view.js  |  2 +-
 4 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/frontend/src/components/dirent-list-view/dirent-list-item.js b/frontend/src/components/dirent-list-view/dirent-list-item.js
index 92aa16c12d..f2aad49bc8 100644
--- a/frontend/src/components/dirent-list-view/dirent-list-item.js
+++ b/frontend/src/components/dirent-list-view/dirent-list-item.js
@@ -431,7 +431,7 @@ class DirentListItem extends React.Component {
                   <li className="operation-group-item">
                     <ItemDropdownMenu
                       item={this.props.dirent}
-                      menuClass={'sf2-icon-caret-down'}
+                      toggleClass={'sf2-icon-caret-down'}
                       isHandleContextMenuEvent={true}
                       getMenuList={this.props.getDirentItemMenuList}
                       onMenuItemClick={this.onMenuItemClick}
@@ -461,7 +461,7 @@ class DirentListItem extends React.Component {
                   <li className="operation-group-item">
                     <ItemDropdownMenu
                       item={this.props.dirent}
-                      menuClass={'sf2-icon-caret-down'}
+                      toggleClass={'sf2-icon-caret-down'}
                       isHandleContextMenuEvent={true}
                       getMenuList={this.props.getDirentItemMenuList}
                       onMenuItemClick={this.onMenuItemClick}
diff --git a/frontend/src/components/dropdown-menu/item-dropdown-menu.js b/frontend/src/components/dropdown-menu/item-dropdown-menu.js
index 60fcd99f2f..ae229a0aab 100644
--- a/frontend/src/components/dropdown-menu/item-dropdown-menu.js
+++ b/frontend/src/components/dropdown-menu/item-dropdown-menu.js
@@ -7,7 +7,7 @@ import { gettext } from '../../utils/constants';
 const propTypes = {
   tagName: PropTypes.string,
   item: PropTypes.object.isRequired,
-  menuClass: PropTypes.string,
+  toggleClass: PropTypes.string,
   isHandleContextMenuEvent: PropTypes.bool,
   getMenuList: PropTypes.func.isRequired,
   onMenuItemClick: PropTypes.func.isRequired,
@@ -19,7 +19,7 @@ class ItemDropdownMenu extends React.Component {
 
   static defaultProps = {
     isHandleContextMenuEvent: true,
-    menuClass: 'sf2-icon-caret-down'
+    toggleClass: 'sf2-icon-caret-down'
   };
 
   constructor(props) {
@@ -94,8 +94,8 @@ class ItemDropdownMenu extends React.Component {
 
   render() {
     let menuList = this.state.menuList;
-    let { menuClass, tagName } = this.props;
-    menuClass = 'sf-dropdown-toggle ' + menuClass;
+    let { toggleClass, tagName } = this.props;
+    toggleClass = 'sf-dropdown-toggle ' + toggleClass;
 
     if (!menuList.length) {
       return '';
@@ -105,7 +105,7 @@ class ItemDropdownMenu extends React.Component {
       return (
         <ButtonDropdown isOpen={this.state.isItemMenuShow} toggle={this.onDropdownToggleClick} title={gettext('More Operations')}>
           <DropdownToggle 
-            className={menuClass}
+            className={toggleClass}
             data-toggle="dropdown" 
             title={gettext('More Operations')}
             aria-expanded={this.state.isItemMenuShow} 
@@ -131,7 +131,7 @@ class ItemDropdownMenu extends React.Component {
       <Dropdown isOpen={this.state.isItemMenuShow} toggle={this.onDropdownToggleClick}>
         <DropdownToggle
           tag={tagName || 'i'}
-          className={menuClass}
+          className={toggleClass}
           title={gettext('More Operations')}
           data-toggle="dropdown" 
           aria-expanded={this.state.isItemMenuShow}
diff --git a/frontend/src/components/toolbar/mutilple-dir-operation-toolbar.js b/frontend/src/components/toolbar/mutilple-dir-operation-toolbar.js
index b00ca5860d..b75373da27 100644
--- a/frontend/src/components/toolbar/mutilple-dir-operation-toolbar.js
+++ b/frontend/src/components/toolbar/mutilple-dir-operation-toolbar.js
@@ -285,7 +285,7 @@ class MutipleDirOperationToolbar extends React.Component {
               <ItemDropdownMenu 
                 tagName={'button'}
                 item={this.props.selectedDirentList[0]}
-                menuClass={'fas fa-ellipsis-v dirents-more-menu'}
+                toggleClass={'fas fa-ellipsis-v dirents-more-menu'}
                 onMenuItemClick={this.onMenuItemClick}
                 getMenuList={this.getDirentMenuList}
               />
diff --git a/frontend/src/components/tree-view/tree-node-view.js b/frontend/src/components/tree-view/tree-node-view.js
index 7d885497d6..4729a973b4 100644
--- a/frontend/src/components/tree-view/tree-node-view.js
+++ b/frontend/src/components/tree-view/tree-node-view.js
@@ -255,7 +255,7 @@ class TreeNodeView extends React.Component {
               {((this.props.repoPermission || permission) && this.state.isShowOperationMenu) && (
                 <ItemDropdownMenu 
                   item={this.props.node}
-                  menuClass={'fas fa-ellipsis-v'}
+                  toggleClass={'fas fa-ellipsis-v'}
                   getMenuList={this.caculateMenuList}
                   onMenuItemClick={this.onMenuItemClick}
                   freezeItem={this.props.freezeItem}