From bc9dda266f57f7ae3a2300a9418d94c3fe62c466 Mon Sep 17 00:00:00 2001 From: Patrick Ziegler Date: Mon, 25 Apr 2022 17:46:53 +0200 Subject: [PATCH] fix(xworkspaces): Update on _NET_DESKTOP_VIEWPORT (#2698) Fixes #2693 --- CHANGELOG.md | 4 +++- src/modules/xworkspaces.cpp | 3 ++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 6f2524d2..05f97753 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -20,7 +20,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - `format-offset` being ignored ([`#2643`](https://github.com/polybar/polybar/pull/2643)) - Negative struts (`margin-bottom`, `margin-top`) being ignored ([`#2642`](https://github.com/polybar/polybar/issues/2642), [`#2644`](https://github.com/polybar/polybar/pull/2644)) - Positioning in awesomeWM ([`#2651`](https://github.com/polybar/polybar/pull/2651)) -- `internal/xworkspaces`: The module sometimes crashed polybar when windows were closed. ([`#2655`](https://github.com/polybar/polybar/pull/2655)) +- `internal/xworkspaces`: + - The module sometimes crashed polybar when windows were closed. ([`#2655`](https://github.com/polybar/polybar/pull/2655)) + - Updates of `_NET_DESKTOP_VIEWPORT` being ignored ([`#2693`](https://github.com/polybar/polybar/issues/2693), [`#2698`](https://github.com/polybar/polybar/pull/2698)) - Mouseover error when only one cursor is defined ([`#2656`](https://github.com/polybar/polybar/pull/2656)) - `custom/script`: Timing inconsistencies ([`#2650`](https://github.com/polybar/polybar/issues/2650), first described at [`#2630`](https://github.com/polybar/polybar/pull/2630)) diff --git a/src/modules/xworkspaces.cpp b/src/modules/xworkspaces.cpp index dd64c44c..9ed49dba 100644 --- a/src/modules/xworkspaces.cpp +++ b/src/modules/xworkspaces.cpp @@ -111,7 +111,8 @@ namespace modules { if (evt->atom == m_ewmh->_NET_CLIENT_LIST || evt->atom == m_ewmh->_NET_WM_DESKTOP) { rebuild_clientlist(); rebuild_desktop_states(); - } else if (evt->atom == m_ewmh->_NET_DESKTOP_NAMES || evt->atom == m_ewmh->_NET_NUMBER_OF_DESKTOPS) { + } else if (evt->atom == m_ewmh->_NET_DESKTOP_NAMES || evt->atom == m_ewmh->_NET_NUMBER_OF_DESKTOPS || + evt->atom == m_ewmh->_NET_DESKTOP_VIEWPORT) { m_desktop_names = get_desktop_names(); rebuild_desktops(); rebuild_clientlist();