From 110bc3e7e9258903230780b25478b03c562cded8 Mon Sep 17 00:00:00 2001
From: Asaad Mahmood <Unknowngi@live.com>
Date: Thu, 29 Oct 2015 00:02:17 +0500
Subject: [PATCH] Multiple UI changes

---
 web/react/components/file_attachment.jsx      |  2 +-
 web/react/components/get_link_modal.jsx       |  2 +-
 web/react/components/navbar_dropdown.jsx      |  1 +
 web/react/components/search_autocomplete.jsx  | 23 ++++++----
 web/react/components/setting_picture.jsx      |  4 +-
 web/react/components/sidebar_header.jsx       | 11 +++++
 .../components/team_signup_with_email.jsx     |  2 +-
 web/react/utils/utils.jsx                     |  2 +
 web/sass-files/sass/partials/_base.scss       |  6 +++
 web/sass-files/sass/partials/_modal.scss      |  5 +--
 web/sass-files/sass/partials/_popover.scss    | 30 +++++++++++++
 web/sass-files/sass/partials/_responsive.scss | 12 +++++-
 web/sass-files/sass/partials/_search.scss     | 42 +------------------
 web/sass-files/sass/partials/_settings.scss   |  3 +-
 14 files changed, 86 insertions(+), 59 deletions(-)

diff --git a/web/react/components/file_attachment.jsx b/web/react/components/file_attachment.jsx
index 4d4e8390c4..e707e32f52 100644
--- a/web/react/components/file_attachment.jsx
+++ b/web/react/components/file_attachment.jsx
@@ -270,7 +270,7 @@ export default class FileAttachment extends React.Component {
                         href={fileUrl}
                         download={filenameString}
                         data-toggle='tooltip'
-                        title={filenameString}
+                        title={'Download ' + filenameString}
                         className='post-image__name'
                     >
                         {trimmedFilename}
diff --git a/web/react/components/get_link_modal.jsx b/web/react/components/get_link_modal.jsx
index 325e86f3db..8839bc3c72 100644
--- a/web/react/components/get_link_modal.jsx
+++ b/web/react/components/get_link_modal.jsx
@@ -98,7 +98,7 @@ export default class GetLinkModal extends React.Component {
                                 <br /><br />
                                 </p>
                                 <textarea
-                                    className='form-control no-resize'
+                                    className='form-control no-resize min-height'
                                     readOnly='true'
                                     ref='textarea'
                                     value={this.state.value}
diff --git a/web/react/components/navbar_dropdown.jsx b/web/react/components/navbar_dropdown.jsx
index 2b68645e5e..2b0f3c40e8 100644
--- a/web/react/components/navbar_dropdown.jsx
+++ b/web/react/components/navbar_dropdown.jsx
@@ -58,6 +58,7 @@ export default class NavbarDropdown extends React.Component {
         TeamStore.addChangeListener(this.onListenerChange);
 
         $(ReactDOM.findDOMNode(this.refs.dropdown)).on('hide.bs.dropdown', () => {
+            $('.sidebar--left .dropdown-menu').scrollTop(0);
             this.blockToggle = true;
             setTimeout(() => {
                 this.blockToggle = false;
diff --git a/web/react/components/search_autocomplete.jsx b/web/react/components/search_autocomplete.jsx
index 03c7b894cf..f7d772677c 100644
--- a/web/react/components/search_autocomplete.jsx
+++ b/web/react/components/search_autocomplete.jsx
@@ -10,6 +10,7 @@ const patterns = new Map([
     ['channels', /\b(?:in|channel):\s*(\S*)$/i],
     ['users', /\bfrom:\s*(\S*)$/i]
 ]);
+const Popover = ReactBootstrap.Popover;
 
 export default class SearchAutocomplete extends React.Component {
     constructor(props) {
@@ -36,6 +37,11 @@ export default class SearchAutocomplete extends React.Component {
         $(document).on('click', this.handleDocumentClick);
     }
 
+    componentDidUpdate() {
+        $(ReactDOM.findDOMNode(this.refs.searchPopover)).find('.popover-content').perfectScrollbar();
+        $(ReactDOM.findDOMNode(this.refs.searchPopover)).find('.popover-content').css('max-height', $(window).height() - 200);
+    }
+
     componentWillUnmount() {
         $(document).off('click', this.handleDocumentClick);
     }
@@ -193,7 +199,7 @@ export default class SearchAutocomplete extends React.Component {
 
         if (this.state.mode === 'channels') {
             suggestions = this.state.suggestions.map((channel, index) => {
-                let className = 'search-autocomplete__channel';
+                let className = 'search-autocomplete__item';
                 if (this.state.selection === index) {
                     className += ' selected';
                 }
@@ -211,7 +217,7 @@ export default class SearchAutocomplete extends React.Component {
             });
         } else if (this.state.mode === 'users') {
             suggestions = this.state.suggestions.map((user, index) => {
-                let className = 'search-autocomplete__user';
+                let className = 'search-autocomplete__item';
                 if (this.state.selection === index) {
                     className += ' selected';
                 }
@@ -224,7 +230,7 @@ export default class SearchAutocomplete extends React.Component {
                         className={className}
                     >
                         <img
-                            className='profile-img'
+                            className='profile-img rounded'
                             src={'/api/v1/users/' + user.id + '/image?time=' + user.update_at}
                         />
                         {user.username}
@@ -234,12 +240,15 @@ export default class SearchAutocomplete extends React.Component {
         }
 
         return (
-            <div
-                ref='container'
-                className='search-autocomplete'
+            <Popover
+                ref='searchPopover'
+                onShow={this.componentDidMount}
+                id='search-autocomplete__popover'
+                className='search-help-popover autocomplete visible'
+                placement='bottom'
             >
                 {suggestions}
-            </div>
+            </Popover>
         );
     }
 }
diff --git a/web/react/components/setting_picture.jsx b/web/react/components/setting_picture.jsx
index b6bcb13a69..e69412cca3 100644
--- a/web/react/components/setting_picture.jsx
+++ b/web/react/components/setting_picture.jsx
@@ -42,7 +42,7 @@ export default class SettingPicture extends React.Component {
             img = (
                 <img
                     ref='image'
-                    className='profile-img'
+                    className='profile-img rounded'
                     src=''
                 />
             );
@@ -50,7 +50,7 @@ export default class SettingPicture extends React.Component {
             img = (
                 <img
                     ref='image'
-                    className='profile-img'
+                    className='profile-img rounded'
                     src={this.props.src}
                 />
             );
diff --git a/web/react/components/sidebar_header.jsx b/web/react/components/sidebar_header.jsx
index de28a8374b..65e4c6d7eb 100644
--- a/web/react/components/sidebar_header.jsx
+++ b/web/react/components/sidebar_header.jsx
@@ -5,6 +5,9 @@ var NavbarDropdown = require('./navbar_dropdown.jsx');
 var UserStore = require('../stores/user_store.jsx');
 const Utils = require('../utils/utils.jsx');
 
+const Tooltip = ReactBootstrap.Tooltip;
+const OverlayTrigger = ReactBootstrap.OverlayTrigger;
+
 export default class SidebarHeader extends React.Component {
     constructor(props) {
         super(props);
@@ -47,7 +50,15 @@ export default class SidebarHeader extends React.Component {
                     {profilePicture}
                     <div className='header__info'>
                         <div className='user__name'>{'@' + me.username}</div>
+                        <OverlayTrigger
+                            trigger={['hover', 'focus']}
+                            delayShow={1000}
+                            placement='bottom'
+                            overlay={<Tooltip id='team-name__tooltip'>{this.props.teamDisplayName}</Tooltip>}
+                            ref='descriptionOverlay'
+                        >
                         <div className='team__name'>{this.props.teamDisplayName}</div>
+                        </OverlayTrigger>
                     </div>
                 </a>
                 <NavbarDropdown
diff --git a/web/react/components/team_signup_with_email.jsx b/web/react/components/team_signup_with_email.jsx
index ff4ccd4d8b..021713f04c 100644
--- a/web/react/components/team_signup_with_email.jsx
+++ b/web/react/components/team_signup_with_email.jsx
@@ -71,7 +71,7 @@ export default class EmailSignUpPage extends React.Component {
                         className='btn btn-md btn-primary'
                         type='submit'
                     >
-                        {'Sign up'}
+                        {'Create Team'}
                     </button>
                     {serverError}
                 </div>
diff --git a/web/react/utils/utils.jsx b/web/react/utils/utils.jsx
index b643c6012c..0d16ef82e6 100644
--- a/web/react/utils/utils.jsx
+++ b/web/react/utils/utils.jsx
@@ -438,6 +438,8 @@ export function applyTheme(theme) {
     if (theme.sidebarTextActiveColor) {
         changeCss('.sidebar--left .nav-pills__container li.active a, .sidebar--left .nav-pills__container li.active a:hover, .sidebar--left .nav-pills__container li.active a:focus, .settings-modal .nav-pills>li.active a, .settings-modal .nav-pills>li.active a:hover, .settings-modal .nav-pills>li.active a:active', 'color:' + theme.sidebarTextActiveColor, 2);
         changeCss('.sidebar--left .nav li.active a, .sidebar--left .nav li.active a:hover, .sidebar--left .nav li.active a:focus', 'background:' + changeOpacity(theme.sidebarTextActiveColor, 0.1), 1);
+        changeCss('.search-help-popover .search-autocomplete__item:hover', 'background:' + changeOpacity(theme.sidebarTextActiveColor, 0.05), 1);
+        changeCss('.search-help-popover .search-autocomplete__item.selected', 'background:' + changeOpacity(theme.sidebarTextActiveColor, 0.15), 1);
     }
 
     if (theme.sidebarHeaderBg) {
diff --git a/web/sass-files/sass/partials/_base.scss b/web/sass-files/sass/partials/_base.scss
index 6399b8fd81..635928fe37 100644
--- a/web/sass-files/sass/partials/_base.scss
+++ b/web/sass-files/sass/partials/_base.scss
@@ -37,6 +37,9 @@ body {
 img {
   max-width: 100%;
   height: auto;
+  &.rounded {
+    @include border-radius(100%);
+  }
 }
 
 .popover {
@@ -122,6 +125,9 @@ a:focus, a:hover {
   &.no-resize {
     resize: none;
   }
+  &.min-height {
+    min-height: 100px;
+  }
 }
 
 .form-control[disabled], .form-control[readonly], fieldset[disabled] .form-control {
diff --git a/web/sass-files/sass/partials/_modal.scss b/web/sass-files/sass/partials/_modal.scss
index 1dcdbf3489..cca57acaab 100644
--- a/web/sass-files/sass/partials/_modal.scss
+++ b/web/sass-files/sass/partials/_modal.scss
@@ -113,11 +113,10 @@
 		text-align: center;
 		padding: 2em 1em;
 		.primary-message {
-			font-size: 1.2em;
+			font-size: 1.25em;
 		}
 		.secondary-message {
-			font-size: 1.25em;
-			color: #888;
+			@include opacity(0.8);
 			margin: 1em 0 0;
 		}
 	}
diff --git a/web/sass-files/sass/partials/_popover.scss b/web/sass-files/sass/partials/_popover.scss
index 4a2ad27485..4f5f1d2158 100644
--- a/web/sass-files/sass/partials/_popover.scss
+++ b/web/sass-files/sass/partials/_popover.scss
@@ -28,6 +28,36 @@
     @include single-transition(opacity, 0.3s, ease-in);
     font-size: em(13px);
 
+    &.autocomplete {
+        display: block;
+        .popover-content {
+            padding: 10px;
+            position: relative;
+        }
+    }
+
+    .search-autocomplete__item {
+        cursor: pointer;
+        padding: 6px 8px;
+        margin: 3px 0;
+        @include border-radius(2px);
+
+        &:hover {
+            background: rgba(black, 0.1);
+        }
+
+        &.selected {
+            background: rgba(black, 0.2);
+        }
+
+        .profile-img {
+            margin-top: -1px;
+            height: 16px;
+            margin-right: 6px;
+            width: 16px;
+        }
+    }
+
     &.bottom > .arrow {
         top: -18px;
         border-width: 9px;
diff --git a/web/sass-files/sass/partials/_responsive.scss b/web/sass-files/sass/partials/_responsive.scss
index 2cd5560efc..b85fa638ab 100644
--- a/web/sass-files/sass/partials/_responsive.scss
+++ b/web/sass-files/sass/partials/_responsive.scss
@@ -379,7 +379,9 @@
             }
             .btn {
                 &.btn-primary {
-                    margin: 8px 0 0 -10px;
+                    display: block;
+                    margin: 10px 0 6px;
+                    width: 100%;
                     float: none;
                 }
             }
@@ -401,7 +403,13 @@
                     &.minimize-settings {
                         display: block;
                         .section-edit {
-                            text-align: left;
+                            position: absolute;
+                            top: 7px;
+                            right: 0;
+                            width: 50px;
+                            .fa {
+                                display: inline-block;
+                            }
                         }
                     }
                     .no-padding--left {
diff --git a/web/sass-files/sass/partials/_search.scss b/web/sass-files/sass/partials/_search.scss
index ce35638854..e50dc398a7 100644
--- a/web/sass-files/sass/partials/_search.scss
+++ b/web/sass-files/sass/partials/_search.scss
@@ -108,44 +108,4 @@
 
 .search-highlight {
 	background-color: #FFF2BB;
-}
-
-.search-autocomplete {
-    background-color: #fff;
-    border: $border-gray;
-    line-height: 36px;
-    overflow-x: hidden;
-    overflow-y: scroll;
-    position: absolute;
-    text-align: left;
-    width: 100%;
-    z-index: 100;
-    @extend %popover-box-shadow;
-}
-
-.search-autocomplete__channel {
-    cursor: pointer;
-    height: 36px;
-    padding: 0px 6px;
-
-    &.selected {
-        background-color:rgba(51, 51, 51, 0.15);
-    }
-}
-
-.search-autocomplete__user {
-    cursor: pointer;
-    height: 36px;
-    padding: 0px;
-
-    .profile-img {
-        height: 32px;
-        margin-right: 6px;
-        width: 32px;
-        @include border-radius(16px);
-    }
-
-    &.selected {
-        background-color:rgba(51, 51, 51, 0.15);
-    }
-}
+}
\ No newline at end of file
diff --git a/web/sass-files/sass/partials/_settings.scss b/web/sass-files/sass/partials/_settings.scss
index d33cffa28c..fbbd074856 100644
--- a/web/sass-files/sass/partials/_settings.scss
+++ b/web/sass-files/sass/partials/_settings.scss
@@ -71,7 +71,7 @@
         .nav {
             position: fixed;
             top: 57px;
-            width: 180px;
+            width: 179px;
         }
         .security-links {
             margin-right: 20px;
@@ -93,6 +93,7 @@
                 padding: 1em 0;
                 margin-bottom: 0;
                 cursor: pointer;
+                position: relative;
                 @include clearfix;
                 &:hover {
                     background: #f9f9f9;