tobast/glowingbear-mainbox
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Only hide bufferlist when switching to mobile

Browse source 
Showing the keyboard on mobile will trigger a resize event, therefore
trying to use the bufferlist filter will actually hide the bufferlist
making the feature unusable. Bug introduced by 554b1669.
This commit is contained in:
Felix Eckhofer 2014-03-30 23:53:29 +02:00
parent 2000437c3a
commit 3ed3df9ede
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
js/glowingbear.js
View file

@ -740,10 +740,13 @@ weechat.controller('WeechatCtrl', ['$rootScope', '$scope', '$store', '$timeout',
var noembed = false; var noembed = false;
var notimestamp = false; var notimestamp = false;
$rootScope.wasMobileDevice = false;
if ($rootScope.isMobileDevice()) { if ($rootScope.isMobileDevice()) {
nonicklist = true; nonicklist = true;
noembed = true; noembed = true;
notimestamp = true; notimestamp = true;
$rootScope.wasMobileDevice = true;
} }
@ -853,10 +856,12 @@ weechat.controller('WeechatCtrl', ['$rootScope', '$scope', '$store', '$timeout',
if ($rootScope.connected) { if ($rootScope.connected) {
// Show the sidebar if switching away from mobile view, hide it when switching to mobile // Show the sidebar if switching away from mobile view, hide it when switching to mobile
// Wrap in a condition so we save ourselves the $apply if nothing changes (50ms or more) // Wrap in a condition so we save ourselves the $apply if nothing changes (50ms or more)
if ($scope.showSidebar === $scope.isMobileDevice()) { if ($scope.wasMobileDevice !== $scope.isMobileDevice() &&
$scope.showSidebar === $scope.isMobileDevice()) {
$scope.showSidebar = !$scope.showSidebar; $scope.showSidebar = !$scope.showSidebar;
$scope.$apply(); $scope.$apply();
} }
$scope.wasMobileDevice = $scope.isMobileDevice();
$scope.calculateNumLines(); $scope.calculateNumLines();
} }
}, 100)); }, 100));