fix order among windows of an application group sometimes not applie…

…d correctly
nclarius · May 27, 2022 · b75e14d · b75e14d
1 parent 7cd212a
commit b75e14d
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 4 deletions.
diff --git a/CHANGELOG.bbcode b/CHANGELOG.bbcode
@@ -1,3 +1,7 @@
+[h2]v1.4[/h2]
+[list]\n[*] fix order among windows of an application group sometimes not applied correctly
+[/list]
+
 [h2]v1.3[/h2]
 [list]\n[*] fix order among windows of an application group sometimes not applied correctly
 [/list]

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,6 @@
+## v1.4
+- fix order among windows of an application group sometimes not applied correctly
+
 ## v1.3
 - fix order among windows of an application group sometimes not applied correctly
 

diff --git a/README.bbcode b/README.bbcode
@@ -1,6 +1,6 @@
 Automatically raises all other visible windows of the same application together when activating one of them, effectively creating application groups to task-switch between.
 
-[b]Please make sure to install the most recent version (v1.3) and to not use Discover for installation.[/b] For more information on installation, setup and usage as well as any requests, please visit [url=https://github.com/nclarius/kwin-application-switcher]the GitHub page[/url].
+[b]Please make sure to install the most recent version (v1.4) and to not use Discover for installation.[/b] For more information on installation, setup and usage as well as any requests, please visit [url=https://github.com/nclarius/kwin-application-switcher]the GitHub page[/url].
 
 This extension gives rise to an application-centric task switching workflow as known from environments such as GNOME or MacOS, where an application’s windows are treated as a group, and task switching can take place at two levels: one mode for switching applications and one mode for switching between windows of an application. 
 

diff --git a/README.md b/README.md
@@ -20,7 +20,7 @@ Seen in the screencast: Switching from Konsole back to Dolphin also brings the o
 
 ### Installation via graphical interface
 
-**Please make sure to select the most recent version (v1.3)** in the installation process.
+**Please make sure to select the most recent version (v1.4)** in the installation process.
 
 A [bug](https://bugs.kde.org/show_bug.cgi?id=453521) in Discover causes a wrong version to be installed, so using the installation module in System Settings instead is recommended.
 

diff --git a/application-switcher_v1.3.kwinscript → application-switcher_v1.4.kwinscript b/application-switcher_v1.3.kwinscript → application-switcher_v1.4.kwinscript
diff --git a/contents/code/main.js b/contents/code/main.js
@@ -114,11 +114,11 @@ workspace.clientActivated.connect(active => {
     debug("previous app", getPrevActiveApp());
     if (getApp(active) != getPrevActiveApp()) {
         debug("app switched");
+        setPrevActiveApp(active);
         // auto-raise other windows of same application
         for (let window of getAppGroup(active)) {
             debug("auto-raising", window.caption);
             workspace.activeClient = window;
         }
-        setPrevActiveApp(active);
     }
 });
diff --git a/metadata.desktop b/metadata.desktop
@@ -8,7 +8,7 @@ Comment=Collectively raises all windows of the same application when switching t
 Icon=preferences-system-windows
 
 X-KDE-PluginInfo-Name=applicationswitcher
-X-KDE-PluginInfo-Version=1.3
+X-KDE-PluginInfo-Version=1.4
 X-KDE-PluginInfo-Author=Natalie Clarius
 X-KDE-PluginInfo-Email[email protected]
 X-KDE-PluginInfo-License=GPLv3.0