summaryrefslogtreecommitdiffstats
path: root/webapp
diff options
context:
space:
mode:
authorenahum <nahumhbl@gmail.com>2017-01-12 17:27:58 -0300
committerHarrison Healey <harrisonmhealey@gmail.com>2017-01-12 15:27:58 -0500
commitaafb8be87c79c60df7534b3b69f967c6301b157e (patch)
tree04ec13efa3b7e94a5493be26076438e731ab7f6a /webapp
parent046d2c61a5fb88742ba232f7d541d9ba9ed42891 (diff)
downloadchat-aafb8be87c79c60df7534b3b69f967c6301b157e.tar.gz
chat-aafb8be87c79c60df7534b3b69f967c6301b157e.tar.bz2
chat-aafb8be87c79c60df7534b3b69f967c6301b157e.zip
Fix ios Team Switch double tap (#5051)
* Fix ios Team Switch double tap * Removing check of the user agent
Diffstat (limited to 'webapp')
-rw-r--r--webapp/components/team_sidebar/components/team_button.jsx20
-rw-r--r--webapp/components/team_sidebar/team_sidebar_controller.jsx14
2 files changed, 20 insertions, 14 deletions
diff --git a/webapp/components/team_sidebar/components/team_button.jsx b/webapp/components/team_sidebar/components/team_button.jsx
index f8b68d5c2..2df21b20b 100644
--- a/webapp/components/team_sidebar/components/team_button.jsx
+++ b/webapp/components/team_sidebar/components/team_button.jsx
@@ -7,9 +7,6 @@ import React from 'react';
import {Link} from 'react-router/es6';
import {Tooltip, OverlayTrigger} from 'react-bootstrap';
-import {isMobile} from 'utils/utils.jsx';
-import {isMobileApp} from 'utils/user_agent.jsx';
-
export default class TeamButton extends React.Component {
constructor(props) {
super(props);
@@ -50,7 +47,14 @@ export default class TeamButton extends React.Component {
</div>
);
}
- if (!isMobile() && !isMobileApp()) {
+ if (this.props.isMobile) {
+ btn = (
+ <div className={'team-btn ' + btnClass}>
+ {badge}
+ {content}
+ </div>
+ );
+ } else {
btn = (
<OverlayTrigger
delayShow={Constants.OVERLAY_TIME_DELAY}
@@ -67,13 +71,6 @@ export default class TeamButton extends React.Component {
</div>
</OverlayTrigger>
);
- } else {
- btn = (
- <div className={'team-btn ' + btnClass}>
- {badge}
- {content}
- </div>
- );
}
return (
@@ -110,6 +107,7 @@ TeamButton.propTypes = {
tip: React.PropTypes.node.isRequired,
active: React.PropTypes.bool,
disabled: React.PropTypes.bool,
+ isMobile: React.PropTypes.bool,
unread: React.PropTypes.bool,
mentions: React.PropTypes.number,
placement: React.PropTypes.oneOf(['left', 'right', 'top', 'bottom'])
diff --git a/webapp/components/team_sidebar/team_sidebar_controller.jsx b/webapp/components/team_sidebar/team_sidebar_controller.jsx
index 24cd48d39..ac3a9f894 100644
--- a/webapp/components/team_sidebar/team_sidebar_controller.jsx
+++ b/webapp/components/team_sidebar/team_sidebar_controller.jsx
@@ -34,7 +34,8 @@ export default class TeamSidebar extends React.Component {
teamListings: TeamStore.getTeamListings(),
teamMembers,
currentTeamId,
- show: teamMembers && teamMembers.length > 1
+ show: teamMembers && teamMembers.length > 1,
+ isMobile: Utils.isMobile()
};
}
@@ -53,12 +54,16 @@ export default class TeamSidebar extends React.Component {
}
componentDidUpdate(prevProps, prevState) {
- $('.team-wrapper').perfectScrollbar();
+ if (!this.state.isMobile) {
+ $('.team-wrapper').perfectScrollbar();
+ }
// reset the scrollbar upon switching teams
if (this.state.currentTeam !== prevState.currentTeam) {
this.refs.container.scrollTop = 0;
- $('.team-wrapper').perfectScrollbar('update');
+ if (!this.state.isMobile) {
+ $('.team-wrapper').perfectScrollbar('update');
+ }
}
}
@@ -121,6 +126,7 @@ export default class TeamSidebar extends React.Component {
url={`/${team.name}`}
tip={team.display_name}
active={team.id === this.state.currentTeamId}
+ isMobile={this.state.isMobile}
displayName={team.display_name}
unread={team.unread}
mentions={team.mentions}
@@ -134,6 +140,7 @@ export default class TeamSidebar extends React.Component {
btnClass='team-btn__add'
key='more_teams'
url='/select_team'
+ isMobile={this.state.isMobile}
tip={
<FormattedMessage
id='team_sidebar.join'
@@ -149,6 +156,7 @@ export default class TeamSidebar extends React.Component {
btnClass='team-btn__add'
key='more_teams'
url='/create_team'
+ isMobile={this.state.isMobile}
tip={
<FormattedMessage
id='navbar_dropdown.create'