Skip to content

Commit

Permalink
Pager: don't restrict new_desk to monitor boundaries
Browse files Browse the repository at this point in the history
When FvwmPager is asked to place a window in its view via a
{new,configurenotify}event, FvwmPager was previously ignoring the
new_desk event if the window's monitor didn't match.

This isn't correct, and the monitor shouldn't be required for tracking a
new desk.

Fixes #433
  • Loading branch information
ThomasAdam committed Feb 19, 2021
1 parent 4eb163c commit 83aca9a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion modules/FvwmPager/FvwmPager.c
Original file line number Diff line number Diff line change
Expand Up @@ -787,7 +787,7 @@ void list_configure(unsigned long *body)
}

is_new_monitor = ((monitor_to_track != NULL) && (t->m != newm));
is_new_desk = (t->desk != cfgpacket->desk && is_new_monitor);
is_new_desk = (t->desk != cfgpacket->desk);

/* If the monitor is different to the one the window was previously on,
* remove the window in the pager as it's no longer on that screen.
Expand Down

0 comments on commit 83aca9a

Please sign in to comment.