Minor UI improvements (#4431)

5 files changed, 59 insertions, 37 deletions

diff --git a/webapp/components/channel_header.jsx b/webapp/components/channel_header.jsx
index 98c6f7947..c7842aaa3 100644
--- a/webapp/components/channel_header.jsx
+++ b/webapp/components/channel_header.jsx
@@ -393,7 +393,7 @@ export default class ChannelHeader extends React.Component {
                     >
                         <FormattedMessage
                             id='channel_header.channelHeader'
-                            defaultMessage='Set Channel Header...'
+                            defaultMessage='Edit Channel Header'
                         />
                     </ToggleModalButton>
                 </li>
@@ -417,6 +417,32 @@ export default class ChannelHeader extends React.Component {
                 </li>
             );
 
+            dropdownContents.push(
+                <li
+                    key='notification_preferences'
+                    role='presentation'
+                >
+                    <ToggleModalButton
+                        role='menuitem'
+                        dialogType={ChannelNotificationsModal}
+                        dialogProps={{
+                            channel,
+                            channelMember: this.state.memberChannel,
+                            currentUser: this.state.currentUser
+                        }}
+                    >
+                        <FormattedMessage
+                            id='channel_header.notificationPreferences'
+                            defaultMessage='Notification Preferences'
+                        />
+                    </ToggleModalButton>
+                </li>
+            );
+
+            dropdownContents.push(
+                <li className='divider'/>
+            );
+
             if (!ChannelStore.isDefault(channel)) {
                 dropdownContents.push(
                     <li
@@ -476,25 +502,7 @@ export default class ChannelHeader extends React.Component {
             }
 
             dropdownContents.push(
-                <li
-                    key='notification_preferences'
-                    role='presentation'
-                >
-                    <ToggleModalButton
-                        role='menuitem'
-                        dialogType={ChannelNotificationsModal}
-                        dialogProps={{
-                            channel,
-                            channelMember: this.state.memberChannel,
-                            currentUser: this.state.currentUser
-                        }}
-                    >
-                        <FormattedMessage
-                            id='channel_header.notificationPreferences'
-                            defaultMessage='Notification Preferences'
-                        />
-                    </ToggleModalButton>
-                </li>
+                <li className='divider'/>
             );
 
             const deleteOption = (
@@ -509,7 +517,7 @@ export default class ChannelHeader extends React.Component {
                     >
                         <FormattedMessage
                             id='channel_header.delete'
-                            defaultMessage='Delete {term}...'
+                            defaultMessage='Delete {term}'
                             values={{
                                 term: (channelTerm)
                             }}
@@ -531,7 +539,7 @@ export default class ChannelHeader extends React.Component {
                         >
                             <FormattedMessage
                                 id='channel_header.setHeader'
-                                defaultMessage='Set {term} Header...'
+                                defaultMessage='Edit {term} Header'
                                 values={{
                                     term: (channelTerm)
                                 }}
@@ -552,7 +560,7 @@ export default class ChannelHeader extends React.Component {
                         >
                             <FormattedMessage
                                 id='channel_header.setPurpose'
-                                defaultMessage='Set {term} Purpose...'
+                                defaultMessage='Edit {term} Purpose'
                                 values={{
                                     term: (channelTerm)
                                 }}
@@ -573,7 +581,7 @@ export default class ChannelHeader extends React.Component {
                         >
                             <FormattedMessage
                                 id='channel_header.rename'
-                                defaultMessage='Rename {term}...'
+                                defaultMessage='Rename {term}'
                                 values={{
                                     term: (channelTerm)
                                 }}
@@ -589,6 +597,10 @@ export default class ChannelHeader extends React.Component {
                 dropdownContents.push(deleteOption);
             }
 
+            dropdownContents.push(
+                <li className='divider'/>
+            );
+
             const canLeave = channel.type === Constants.PRIVATE_CHANNEL ? this.state.userCount > 1 : true;
             if (!ChannelStore.isDefault(channel) && canLeave) {
                 dropdownContents.push(
diff --git a/webapp/components/delete_channel_modal.jsx b/webapp/components/delete_channel_modal.jsx
index caff45aae..7ad213395 100644
--- a/webapp/components/delete_channel_modal.jsx
+++ b/webapp/components/delete_channel_modal.jsx
@@ -69,14 +69,16 @@ export default class DeleteChannelModal extends React.Component {
                     </h4>
                 </Modal.Header>
                 <Modal.Body>
-                    <FormattedMessage
-                        id='delete_channel.question'
-                        defaultMessage='This will delete the channel from the team and make its contents inaccessible for all users. Are you sure you wish to delete the {display_name} {term}?'
-                        values={{
-                            display_name: this.props.channel.display_name,
-                            term: (channelTerm)
-                        }}
-                    />
+                    <div className='alert alert-danger'>
+                        <FormattedMessage
+                            id='delete_channel.question'
+                            defaultMessage='This will delete the channel from the team and make its contents inaccessible for all users. Are you sure you wish to delete the {display_name} {term}?'
+                            values={{
+                                display_name: this.props.channel.display_name,
+                                term: (channelTerm)
+                            }}
+                        />
+                    </div>
                 </Modal.Body>
                 <Modal.Footer>
                     <button
diff --git a/webapp/i18n/en.json b/webapp/i18n/en.json
index f6b187d44..20f771608 100644
--- a/webapp/i18n/en.json
+++ b/webapp/i18n/en.json
@@ -981,8 +981,8 @@
   "channel_flow.invalidName": "Invalid Channel Name",
   "channel_flow.set_url_title": "Set {term} URL",
   "channel_header.channel": "Channel",
-  "channel_header.channelHeader": "Set Channel Header...",
-  "channel_header.delete": "Delete {term}...",
+  "channel_header.channelHeader": "Edit Channel Header",
+  "channel_header.delete": "Delete {term}",
   "channel_header.flagged": "Flagged Posts",
   "channel_header.group": "Group",
   "channel_header.leave": "Leave {term}",
@@ -991,9 +991,9 @@
   "channel_header.manageMembers": "Manage Members",
   "channel_header.notificationPreferences": "Notification Preferences",
   "channel_header.recentMentions": "Recent Mentions",
-  "channel_header.rename": "Rename {term}...",
-  "channel_header.setHeader": "Set {term} Header...",
-  "channel_header.setPurpose": "Set {term} Purpose...",
+  "channel_header.rename": "Rename {term}",
+  "channel_header.setHeader": "Edit {term} Header",
+  "channel_header.setPurpose": "Edit {term} Purpose",
   "channel_header.viewInfo": "View Info",
   "channel_header.viewMembers": "View Members",
   "channel_header.webrtc.call": "Start Video Call",
diff --git a/webapp/sass/components/_dropdown.scss b/webapp/sass/components/_dropdown.scss
index 0051f09a3..be68713ff 100644
--- a/webapp/sass/components/_dropdown.scss
+++ b/webapp/sass/components/_dropdown.scss
@@ -15,6 +15,10 @@
 
     .divider {
         @include opacity(.15);
+
+        &:last-child {
+            display: none;
+        }
     }
 
     div,
diff --git a/webapp/sass/components/_modal.scss b/webapp/sass/components/_modal.scss
index 71ed8e693..7a44e3b96 100644
--- a/webapp/sass/components/_modal.scss
+++ b/webapp/sass/components/_modal.scss
@@ -73,6 +73,10 @@
             }
         }
 
+        .alert {
+            margin-bottom: 0;
+        }
+
         .custom-textarea {
             border-color: $light-gray;
             color: inherit;