THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added:
FS#699 - hiding toplevel windows is not supported User who did this - Uli Schlachter (psychon) ---------- Ok, I'm a little confused on a way to go here. My latest commit (which sadly was already merged) definitely is buggy and has to be removed ASAP (@jd: Feel free to revert). To fix this bug we have to xcb_unmap_window() in the UnmapNotify handle right after calling client_unmanage(). But we should also trigger the switch to withdrawn state if a client just unmaps one of its windows according to ICCCM (this should also fix some other bugs where windows still show up on the tasklist even though they shouldn't). The first part is easy: As I said, just add a call to xcb_unmap_window() (I verified this to actually work). This last part is where it's getting tricky. We have to client_unmanage() on a real UnmapNotify. But not if this event is from awesome, else we lose clients on tag switch (this is where my first patch failed badly, oh and I fucked up an if). So how do we find out if it was awesome who generated the unmap or a client? If we just ignored real unmap notifies as we did before, how many apps would break? ---------- More information can be found at the following URL: http://awesome.naquadah.org/bugs/index.php?do=details&task_id=699#comment1695 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above. -- To unsubscribe, send mail to [email protected].
