From cb4dedfb15e9855b76abd7ad345af2d885c59208 Mon Sep 17 00:00:00 2001
From: Michael An <2331806369@qq.com>
Date: Thu, 7 Mar 2019 17:14:03 +0800
Subject: [PATCH] fix style (#3077)

---
 .../components/dialog/add-reviewer-dialog.js  | 25 ++++++++++---------
 .../dialog/manage-members-dialog.js           |  2 +-
 frontend/src/css/add-reviewer-dialog.css      | 10 ++++++++
 3 files changed, 24 insertions(+), 13 deletions(-)

diff --git a/frontend/src/components/dialog/add-reviewer-dialog.js b/frontend/src/components/dialog/add-reviewer-dialog.js
index 3bfac5a750..027e358102 100644
--- a/frontend/src/components/dialog/add-reviewer-dialog.js
+++ b/frontend/src/components/dialog/add-reviewer-dialog.js
@@ -96,13 +96,19 @@ class AddReviewerDialog extends React.Component {
         <ModalHeader>{gettext('Request a review')}</ModalHeader>
         <ModalBody >
           <p>{gettext('Add new reviewer')}</p>
-          <UserSelect
-            placeholder={gettext('Please enter 1 or more character')}
-            onSelectChange={this.handleSelectChange}
-            ref="reviewSelect"
-            isMulti={true}
-            className='reviewer-select'
-          />
+          <div className='add-reviewer'>
+            <UserSelect
+              placeholder={gettext('Please enter 1 or more character')}
+              onSelectChange={this.handleSelectChange}
+              ref="reviewSelect"
+              isMulti={true}
+              className='reviewer-select'
+            />
+            {(this.state.selectedOption && !this.state.loading)?
+              <Button color="secondary" onClick={this.addReviewers}>{gettext('Submit')}</Button> :
+              <Button color="secondary" disabled>{gettext('Submit')}</Button>
+            }
+          </div>
           {this.state.errorMsg.length > 0 &&
             this.state.errorMsg.map((item, index = 0, arr) => {
               return (
@@ -126,11 +132,6 @@ class AddReviewerDialog extends React.Component {
           }
         </ModalBody>
         <ModalFooter>
-          { this.state.loading ?
-            <Button disabled><i className="fa fa-spinner" aria-hidden="true"></i></Button>
-            :
-            <Button color="primary" onClick={this.addReviewers}>{gettext('Submit')}</Button>
-          }
           <Button color="secondary" onClick={this.props.toggleAddReviewerDialog}>
             {gettext('Close')}</Button>
         </ModalFooter>
diff --git a/frontend/src/components/dialog/manage-members-dialog.js b/frontend/src/components/dialog/manage-members-dialog.js
index 1620b4d412..315d225037 100644
--- a/frontend/src/components/dialog/manage-members-dialog.js
+++ b/frontend/src/components/dialog/manage-members-dialog.js
@@ -86,7 +86,7 @@ class ManageMembersDialog extends React.Component {
           <p>{gettext('Add group member')}</p>
           <div className='add-members'>
             <UserSelect
-              placeholder={gettext('Search users...')}
+              placeholder={gettext('Select users...')}
               onSelectChange={this.onSelectChange}
               ref="userSelect"
               isMulti={true}
diff --git a/frontend/src/css/add-reviewer-dialog.css b/frontend/src/css/add-reviewer-dialog.css
index bc77ad859c..a2aff01333 100644
--- a/frontend/src/css/add-reviewer-dialog.css
+++ b/frontend/src/css/add-reviewer-dialog.css
@@ -25,4 +25,14 @@
   cursor: pointer;
   opacity: 1;
   color: #a4a4a4;
+}
+.add-reviewer {
+  display: flex;
+  justify-content: space-between;
+}
+.add-reviewer .reviewer-select {
+  width: 385px;
+}
+.add-reviewer .btn {
+  width: 75px;
 }
\ No newline at end of file