summary refs log tree commit diff
Commit message (Collapse)AuthorAgeFilesLines
* If not using sticky mode (the default), clients aren't automagically cn-bisectgtkbugokan2013-12-082-3/+3
| | | | | assigned a group, thus cc->group will be NULL - fix the client group shortcut in menu lists; crash reported by Christian Neukirchen.
* When we receive a NotionNotify event, there's no need to (incorrectlyokan2013-12-021-3/+0
| | | | might I add) redraw the top menu selection.
* Always highlight the first menu item (helpful to see selected items whenokan2013-12-021-4/+2
| | | | | not using a mouse and therefore not generating Expose/MotionNotify events); from Thomas Adam.
* Prepend the group shortcut in the client search menu; from Thomas Adam.okan2013-12-022-3/+4
| | | | Likewise, prepend shortcut in unhide menu.
* Since we use IconicState as our hidden state, check for the hidden flagokan2013-12-021-2/+4
| | | | | | | | and unmanage the client if we're not hidden (basically if NormalState) during an UnmapNotify event. Resolves an issue with mplayer going fullscreen while not using NetWM hints; behaviour regression reported by Ido Admon.
* like gap, make snapdist per screenokan2013-11-274-4/+6
|
* alter -r1.145 getsizehints to deal with clients that don't haveokan2013-11-271-35/+29
| | | | WM_NORMAL_HINTS.
* Remove the option to bind a key by keycode with brackets; it neverokan2013-11-273-22/+7
| | | | | | | worked (and no one complained!). While it's fairly easy to fix, users should be using keysym names and not keycodes. Discussed at length months ago with todd@, matthieu@ and Owain.
* simplify/unfoldokan2013-11-272-11/+7
|
* Rewrite active/inactive client handling in client_setactive();okan2013-11-273-70/+35
| | | | | | | | | | | client_leave() served no real purpose, likewise no reason to handle LeaveNotify events since an EnterNotify will process the next active client (and we don't have anything important to process anyway), so xev_handle_leavenotify() goes as well. Allows a simplification of client_mtf() and client_cycle_leave() for clarity. While here, unify a few client_current() checks. No intended behaviour change.
* Log message:benno2013-11-251-5/+5
| | | | | | use u_char for buffers in yylex, for ctype calls, as done in all other parse.y in the tree found by millert@, ok okan@
* Alter the r1.35 of event.c race fix. Remove the forward looking eventokan2013-11-123-27/+6
| | | | | | | | | | | | queue check (removing the need for a server grab/ungrab) - if the client is going away, let it fall all the way through to a DestroyNotify event. There's no longer a need for us to manually destroy a client ourselves (removing yet another server grab/ungrab). Instead, when the UnmapNotify event is synthetic, simply set the state to Withdrawn (as per ICCCM), else Iconic (in our case 'hidden'). Verified with test case from the 2009 race which was the original reason for r1.35 of event.c.
* Put back the border draw call in client_resize; since we are adding andokan2013-11-111-0/+2
| | | | | | removing the border on maximized clients we need to redraw. Also noticed by Tim van der Molen
* stash WMProtocols in flagsokan2013-11-082-11/+7
|
* quick keyboard focus fix for clients that neither populate wmhints nor ↵okan2013-11-051-1/+2
| | | | wmprotocols, like rdesktop; focus needs to be re-visited
* The only reason we need to keep XSizeHints in our client_ctx is forokan2013-11-022-31/+38
| | | | | flags, so just add one to hints; consolidates sizehints and shrinks. Additionally don't abuse PSize for XGetWMNormalHints() failures.
* x/y from XSizeHints are obsolete (and have been for a long time), sookan2013-11-011-4/+2
| | | | | instead use x/y from XWindowAttributes when USPosition|PPosition are set.
* re-add support for WM_TAKE_FOCUS, and additionally this time only callokan2013-11-012-8/+18
| | | | | | XSetInputFocus() for clients that have the InputHint; latter fix discovered by Valery Masiutsin with a PoC patch - solves keyboard input focus loss for java apps.
* A client_delete should behave differently depending on whether theokan2013-10-253-10/+12
| | | | | | | triggering event was unmap (with pending destroy) log destroy; we only need to grab/ungrab the server lock, set WithdrawnState and XRemoveFromSaveSet clients coming from an unmap event - doing so for clients coming from destroy are already gone, hence we generate errors.
* shuffleokan2013-10-201-37/+37
|
* wrap key ungrab like btn ungrab, for it'll be used againokan2013-10-203-1/+8
|
* revert 1.138 (WM_TAKE_FOCUS) for nowokan2013-10-201-2/+0
|
* Using xu_btn_ungrab() buttons during client_leave doesn't work (errorokan2013-10-194-11/+5
| | | | | | | BadValue) when the modifier is already AnyModifier . Instead alter xu_btn_ungrab() to ungrab AnyButton/AnyModifier and call it only when a client is coming into focus in client_setactive(), instead of iterating over ignore mods - matches how we handle key grabs.
* I believe we redraw the border in too many cases; likely a leftover fromokan2013-10-192-11/+2
| | | | | | | | | the cc->pwin days - don't redraw on every unhide, resize, and mouse move/resize Expose event (note, all Expose events trigger a redraw anyway). Tested with some finicky X apps I could think of, though I'm sure others will find more - if so, and they 'lose' the border, please report!
* For clients that support WM_TAKE_FOCUS in their WM_PROTOCOLS property, sendokan2013-10-191-0/+2
| | | | a ClientMessage event.
* a few err->errx since we don't have error messages here; from Tiago Cunhaokan2013-10-172-5/+5
|
* trying parsing a XLFD string first, then by patternokan2013-10-071-3/+6
|
* unify type; no changeokan2013-10-034-6/+6
|
* we no longer need to construct an array for _NET_SUPPORTED.okan2013-07-161-7/+1
|
* don't need the size hereokan2013-07-161-1/+1
|
* max -> nitemsokan2013-07-163-5/+5
|
* shuffle down protoypesokan2013-07-151-50/+47
|
* collapse linesokan2013-07-153-10/+6
|
* simplify atom handling; allows us to limit to one round-trip to serverokan2013-07-157-94/+88
| | | | for gathering Atoms.
* bring buttonpress and keypress event handlers slightly closer togetherokan2013-07-101-8/+7
|
* type fixesokan2013-07-102-3/+3
|
* since the root and event window are the same in the case of a buttonokan2013-07-093-6/+3
| | | | | | | event on the screen's root window, there's no need to pass down the entire XButtonEvent event, at least to group_menu(), the only callback which takes an argument at this point; instead use the already populated screen.
* add support for mouse based group {,r}cycle; from Rodrigo Mosconi. notokan2013-07-084-1/+23
| | | | bound by default.
* move kbfunc and mousefunc closer togetherokan2013-07-085-52/+56
|
* clarify kbd vs mouse functionsokan2013-07-083-20/+20
|
* move duplicate kbd and mouse modifier parsing to a generic function;okan2013-07-081-34/+30
| | | | from Tiago Cunha
* replace a few (x)malloc with (x)calloc to prevent potential integerokan2013-07-082-5/+5
| | | | overflows; from Tiago Cunha
* whitespace and style fixes; from Tiago Cunha and one from me.okan2013-07-082-4/+2
|
* properly fix (and re-organize) selfont selection.okan2013-06-231-13/+11
|
* when selfont is configured, make sure we continue and configure the restokan2013-06-201-2/+2
| | | | of the screen (quick fix); discovered the hard way by Rodrigo Mosconi.
* move Cursors into conf.okan2013-06-176-24/+36
|
* now that we have conf_screen, which configures individual screensokan2013-06-171-12/+5
| | | | | *after* config parsing, we no longer need to split up display/screen initialization, so collapse.
* allow mouse button4 and button5; from Rodrigo Mosconiokan2013-06-172-3/+9
|
* move synthetic responses and have client_msg only work with WM_PROTOCOLS,okan2013-06-104-43/+43
| | | | since that's all ClientMessageEvent is for anyway.
* Check for, and honour, CWStackMode and CWSibling change requests during aokan2013-06-101-1/+5
| | | | | ConfigureRequest event. Additionally, honour a border width change; detailed report and patch from Mike Small.