At this point, desktop() is -1 since the client hasn't been moved to any
virtual desktop yet.

It seems like checking desktop() is a typo. It makes more sense to check
desktopId.
remotes/origin/work/abstract-egl-backend-no-surface
Vlad Zahorodnii 5 years ago
parent 7f04d730e6
commit bc7b28bb65
  1. 2
      src/x11client.cpp

@ -548,7 +548,7 @@ bool X11Client::manage(xcb_window_t w, bool isMapped)
if (info->desktop()) { if (info->desktop()) {
desktopId = info->desktop(); // Window had the initial desktop property, force it desktopId = info->desktop(); // Window had the initial desktop property, force it
} }
if (desktop() == 0 && asn_valid && asn_data.desktop() != 0) { if (desktopId == 0 && asn_valid && asn_data.desktop() != 0) {
desktopId = asn_data.desktop(); desktopId = asn_data.desktop();
} }
if (desktopId) { if (desktopId) {

Loading…
Cancel
Save