aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* gc->hidden has never consistently kept track of a group's state;okan2014-08-24-35/+21
| | | | | | | | | | group_show() and group_hide() are not the only ways a group can change state - if all clients in a group are either hidden or unhidden, then that group's state should change, as well as the various EWMH ways. Instead of trying to keep track in a wide variety of places, simply query the clients in a group before needing to take action based on the group's state. Solves long standing confusion of when a group is hidden or not.
* Fix nogroup regression, where nogroup became an actual group - theokan2014-08-22-19/+22
| | | | | | | | | | symantics between cwm groups and ewmh got in the way. Ensure a client that wants to be in nogroup stays in nogroup (thus stays in view), even when (re)reading NET_WM_DESKTOP. Paritially reverts patchset 644 (2014-02-07 13:09 PST) which deals with a NULL cc->group. All to be revisited when NET_WM_STATE_STICKY hits cwm. Reported by many; testing and ok phessler.
* Purely mechanical; unify 'num', 'no' and 'shortcut'.okan2014-08-20-39/+38
|
* Split off group window restacking.okan2014-08-20-5/+16
|
* whitespaceokan2014-08-20-2/+2
|
* fix a misleading comment and function nameokan2014-08-20-6/+6
|
* Remove unnecessary memset since reload was replaced with restart;okan2014-08-19-3/+1
| | | | from Kent Spillner.
* Pull highstack from group_ctx (and useless calculations of); in the oneokan2014-08-19-19/+11
| | | | | place that we use highstack, replace that usage with a local variable (for now until stacking is done properly).
* Get rid of nhidden in group_ctx; it actually never reported correctlyokan2014-08-18-10/+7
| | | | | | | since nhidden wasn't incremented nor decremeted in all the right places, thus confusing matters. We don't need to carry a count around, so just use a local variable in the one place we need one to supply XRestackWindows().
* plug memleak (currently unused func); from Tiago Cunhaokan2014-04-21-1/+2
|
* Alignment fix; found the hard way on sparc64 by landry@ (and thanks toookan2014-02-27-2/+2
| | | | | | for fast access to a box!) ok deraadt@, matthieu@
* Make sure we cycle through existing client's group membership and setokan2014-02-08-3/+14
| | | | | the hidden state on the group; important to know group hidden state after a re-exec or restart of cwm (as easily seen in group menu).
* Replace a few hand rolled loops with like tailq loops.okan2014-02-08-17/+13
|
* all mapped clients now should have a group, so simplify some casesokan2014-02-07-10/+6
|
* If _NET_WM_DESKTOP is set to -1 during client creation, place the client intookan2014-02-07-29/+13
| | | | | | | | | | | | group 0 (nogroup); solves problem initially discovered by oga@nicotinebsd with tint2. A clientmessage *after* client creation already handles this case. Go further and assign every client to a group; in non-sticky mode, group 0 (nogroup) and sticky mode, the active group. In both cases, autogroup will override the group assignment. Removing a group from a client always places the client back into group 0 (nogroup). Autogroup can also assign a client to group 0 (nogroup) to keep a client always visible (unless of course one opts to hide all clients).
* Some clients set the urgency flag even if they are the active client;okan2014-02-06-2/+3
| | | | | prevent annoying behavior by only setting the cwm urgency flag if the client is not active; diff from Thomas Adam.
* Introduce a region queue and replace screen's XineramaScreenInfo; weokan2014-02-03-26/+36
| | | | | still use Xinerama to populate the regions per screen, but will switch at a more appropriate time.
* make this XQueryTree like the otherokan2014-02-03-13/+11
|
* Move redundant bits from screen_init (while dealing with existingokan2014-02-03-19/+21
| | | | | | clients) directly into client_init, performing the X roundtrip only once. With the previous change in maprequest, this moves decision making into one place for creating new clients.
* Move redundant window attr fetch from maprequest directly intookan2014-02-02-9/+9
| | | | client_init and perform that X roundtrip only once.
* No need to store screen colormap and visual; rather just use the Xokan2014-02-02-15/+13
| | | | macros in the one place they are needed.
* Unlike free(3), XFree(3) can't be passed a NULL pointer.okan2014-02-02-3/+3
|
* The menu already limits entries with MENU_MAXENTRY, so don't botherokan2014-02-02-11/+10
| | | | holding a command name limit as well.
* Take menu's bwidth into account for position/size; reported by Thomas Adam.okan2014-02-01-3/+3
|
* Let the config parser continue parsing even after encountering an error;okan2014-02-01-13/+20
| | | | | | | | original idea (with now-outdated patch) from Thomas Adam. Since we now report where errors exist, this now makes more sense. Expand a bit on config file parsing in the cwm(1). Discussed with a few, including sthen; ok sthen.
* re-add lost free from previousokan2014-01-30-1/+2
|
* Switch ignoreq to winname struct since it's basically the same thing;okan2014-01-30-36/+20
| | | | removes limit on name matching.
* use the same autogroupwin variable as everywhere elseokan2014-01-30-7/+7
|
* Move cwm_status around to avoid header fu, and remove extraneous signalokan2014-01-30-5/+5
| | | | header from xevents.c; noticed by Christian Neukirchen.
* Remove duplicate mouse functions and instead use the kbfunc ones. Nookan2014-01-30-41/+8
| | | | | user visable changes at this point, but they'll merge at an appropriate time.
* Minimize trivial differences between a few kb and mb functions.okan2014-01-29-14/+13
|
* keybinding -> key bindingokan2014-01-29-8/+8
|
* Merge keybinding and mousebinding queues into using the same mergedokan2014-01-29-46/+41
| | | | | struct, binding; they were essentially the same accept for what was 'pressed', keysym or button.
* Much like we do for keyboard and mouse bindings, remove duplicates forokan2014-01-29-1/+16
| | | | command name - last match.
* Check command name/path for truncation and provide user feedback duringokan2014-01-29-18/+33
| | | | | | config parse (and use conf_cmd_add to populate defaults); based on a discussion with Tiago Cunha. While this looks ugly, there are likely some other changes here to come.
* Check ignore windowname for truncation and provide user feedback duringokan2014-01-28-7/+13
| | | | config parse; based on a discussion with Tiago Cunha.
* The EWMH spec states that if the cardinal returned is 0xFFFFFFFF (-1)okan2014-01-28-3/+12
| | | | | | then the window should appear on all desktops, which in our case is assigned to group 0. Found to fix stalonetray due to the non-ewmh aware range checking in group_movetogroup(); from Thomas Adam.
* Move conf_init/clear into main - no behaviour change; from Tiago Cunha.okan2014-01-28-9/+7
|
* move some init up and shed some blank linesokan2014-01-27-8/+5
|
* simplify parse_config using assignment; inspired by a very old diff fromokan2014-01-27-58/+3
| | | | Tiago Cunha.
* more range checkingokan2014-01-24-5/+8
|
* correct arguments and drop the cast.okan2014-01-24-2/+2
| | | | sanity check by oga@nicotinebsd
* no need to quote within warningokan2014-01-23-3/+3
|
* If no title is supplied, term uses only the basename for its title.okan2014-01-23-3/+3
| | | | | | | This is useless when searching for windows launched via the ssh command menu; supply a more useful title: '[ssh] <hostname>'. Idea from todd@, ok todd@
* zap whitespaceokan2014-01-22-26/+26
|
* cwm_argv doesn't need to be global any longerokan2014-01-22-6/+6
|
* start properly releasing X resources during teardownokan2014-01-22-1/+20
|
* Somewhat streamline event loop/restart/quit handling; most notableokan2014-01-22-33/+36
| | | | | | | | change allows a restart to trigger proper teardown first, even though teardown is not (yet) complete. After some discussion with oga@nicotinebsd.org regarding a more complicated version/idea.
* Sprinkle a few more const; from Tiago Cunha.okan2014-01-21-22/+23
|
* Save the ptr position before lowering via kbd, so as to be able to cycleokan2014-01-20-1/+2
| | | | | back with the pointer in the right place; matches behaviour when lowering via the mouse function.