From 8b97a0696184ffb2ea36ff2e079615bbf594b3d1 Mon Sep 17 00:00:00 2001 From: Georges Basile Stavracas Neto Date: Tue, 6 Aug 2019 19:41:54 -0300 Subject: [PATCH] allView: Always update currentPage Commit 0f178c3b3dcfe added a shortcirtuit to avoid running an animation on an invisible actor. However, it introduced a bug where the current page is not properly updated. That leads to the wrong set of icons being animated under some circumstances. Update the current page even if we bail out early. https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/667 --- js/ui/appDisplay.js | 1 + 1 file changed, 1 insertion(+) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index 5fe140949..9947fdcae 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -467,6 +467,7 @@ var AllView = class AllView extends BaseAppView { if (!this.actor.mapped) { this._adjustment.value = this._grid.getPageY(pageNumber); this._pageIndicators.setCurrentPage(pageNumber); + this._grid.currentPage = pageNumber; return; }