Commit Diff

Commit:
dce4fb373757727374d00c857ec0dfd225bbeafd
From:
Quentin Rameau <quinq+hackers@fifth.space>
Via:
Hiltjo Posthuma <hiltjo@codemadness.org>
Date:
Message:
setfullscreen: don't process the property twice Some clients try to set _NET_WM_STATE_FULLSCREEN even when the window is already in fullscreen. For example, c->oldstate was set two times in a raw and window would then always be floating. We must check that it's not the case before processing it. (original patch modified with suggestion from Markus Teich <markus.teich@stusta.mhn.de>)
Actions:
Patch | Tree
commit - 646b351cc79845f4cc77415dfff474b9ae0053d9
commit + dce4fb373757727374d00c857ec0dfd225bbeafd
blob - 96b43f7edfb2a01cd49e891bcfa5966a3057d687
blob + 36391655ec65b873a222e1e415b3e60e6227df08
--- dwm.c
+++ dwm.c
@@ -1441,7 +1441,7 @@ setfocus(Client *c) {
 
 void
 setfullscreen(Client *c, Bool fullscreen) {
-	if(fullscreen) {
+	if(fullscreen && !c->isfullscreen) {
 		XChangeProperty(dpy, c->win, netatom[NetWMState], XA_ATOM, 32,
 		                PropModeReplace, (unsigned char*)&netatom[NetWMFullscreen], 1);
 		c->isfullscreen = True;
@@ -1452,7 +1452,7 @@ setfullscreen(Client *c, Bool fullscreen) {
 		resizeclient(c, c->mon->mx, c->mon->my, c->mon->mw, c->mon->mh);
 		XRaiseWindow(dpy, c->win);
 	}
-	else {
+	else if(!fullscreen && c->isfullscreen){
 		XChangeProperty(dpy, c->win, netatom[NetWMState], XA_ATOM, 32,
 		                PropModeReplace, (unsigned char*)0, 0);
 		c->isfullscreen = False;