PLT-4343 Fixes for mobile main menu (#4148)

* Fixed mobile app link in the main menu to be displayed on mobile browsers

* Fixed doubled up dividers in mobile menu

* Added scrolling to mobile main menu
Этот коммит содержится в:
Harrison Healey
2016-10-04 14:38:19 -04:00
коммит произвёл GitHub
родитель b9dc759449
Коммит 816a738d28
10 изменённых файлов: 20 добавлений и 12 удалений

Просмотреть файл

@@ -911,7 +911,7 @@ export function getSuggestedCommands(command, suggestionId, component) {
(data) => {
var matches = [];
data.forEach((cmd) => {
if (cmd.trigger !== 'shortcuts' || !UserAgent.isMobileApp()) {
if (cmd.trigger !== 'shortcuts' || !UserAgent.isMobile()) {
if (('/' + cmd.trigger).indexOf(command) === 0) {
const s = '/' + cmd.trigger;
let hint = '';

Просмотреть файл

@@ -77,8 +77,15 @@ export function isAndroidWeb() {
return isAndroidChrome();
}
// Returns true if and only if the user is using a Mattermost mobile app. This will return false if the user is using the
// web browser on a mobile device.
export function isMobileApp() {
return isAndroid() || isIos();
return userAgent.indexOf('iPhone') !== -1 && userAgent.indexOf('Safari') === -1 && userAgent.indexOf('CriOS') === -1;
}
// Returns true if and only if the user is using Mattermost from either the mobile app or the web browser on a mobile device.
export function isMobile() {
return isIos() || isAndroid();
}
export function isFirefox() {