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

Fix "Switch to buffer with activity" skipping buffers with notifications

Browse source 
This caused two buffer switches in a row, in effect forgetting the notifications
This commit is contained in:
Lorenz Hübschle-Schneider 2014-02-21 16:03:35 +00:00
parent f10e44efe4
commit d4a5218922
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
js/glowingbear.js
View file

@ -832,7 +832,7 @@ weechat.controller('WeechatCtrl', ['$rootScope', '$scope', '$store', '$timeout',
buffer = sortedBuffers[i]; buffer = sortedBuffers[i];
if (buffer.notification > 0) { if (buffer.notification > 0) {
$scope.setActiveBuffer(buffer.id); $scope.setActiveBuffer(buffer.id);
break; return; // return instead of break so that the second for loop isn't executed
} }
} }
// No notifications, find first buffer with unread lines instead // No notifications, find first buffer with unread lines instead
@ -840,7 +840,7 @@ weechat.controller('WeechatCtrl', ['$rootScope', '$scope', '$store', '$timeout',
buffer = sortedBuffers[i]; buffer = sortedBuffers[i];
if (buffer.unread > 0) { if (buffer.unread > 0) {
$scope.setActiveBuffer(buffer.id); $scope.setActiveBuffer(buffer.id);
break; return;
} }
} }
}; };