| Commit message (Collapse) | Author | Age | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This functionally partly reverts commit db0f56a (Do not warp mouse to
windows marked active by EWMH, 2022-01-17) which introduced a bug where
the pointer would no longer be warped to windows activated by rofi.
Since the window switcher internal to cwm calls client_ptr_warp directly
this was not noticed immediately.
Thankfully the EWMH spec defines a "source indication" [1] in the
_NET_ACTIVE_WINDOW request [2]. This field specifies whether the request
came from a normal application or a pager and "other clients that
represent direct user actions". Window switchers like rofi should honor
this field and therefore it is trivial to fix this bug by checking for
the correct source indication value.
This means that normal windows are now prohibited from warping the
pointer under any circumstance, but the user (through specific
applications) may still elect to have the window manager honor
_NET_ACTIVE_WINDOW this way.
[1] https://specifications.freedesktop.org/wm-spec/wm-spec-1.3.html#sourceindication
[2] https://specifications.freedesktop.org/wm-spec/wm-spec-1.3.html#idm45682917892304
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As discussed a couple of times [1] [2] on the openbsd-* mailing lists,
modal dialogs that are marked as active through EWMH will have cwm warp
the mouse back to them once it leaves the constraints of the window.
For us this is particularly a problem with dialogs created by QT (such
as the settings dialog in Quassel). Since no real solution was ever
posted on the mailing lists, just disable the warping behaviour for now.
This should not have any detrimental impact other than for the dialogs
that are "misbehaving" right now.
[1] https://marc.info/?l=openbsd-misc&m=155542694605428&w=2
[2] https://marc.info/?l=openbsd-ports&m=155508625010715&w=2
|
|
|
|
|
|
|
|
|
|
| |
The previous commit as taken from upstream introduces a compiler warning
for parse.y:
parse.y:110:42: warning: format ‘%lld’ expects argument of
type ‘long long int’, but argument 3 has type ‘int64_t’
Quelch this warning by using PRId64 from inttypes.h
|
|
|
|
| |
other parse.y's; from Leon Fischer <lfischer@airmail.cc>.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
original from naddy@:
> Don't declare variables as "unsigned char *" that are passed to
> functions that take "char *" arguments. Where such chars are
> assigned to int or passed to ctype functions, explicitly cast them
> to unsigned char.
>
> For OpenBSD's clang, -Wpointer-sign has been disabled by default,
> but when the parse.y code was built elsewhere, the compiler would
> complain.
>
> With help from millert@
> ok benno@ deraadt@
|
|
|
|
|
|
| |
keys support.
found and fix by Luis Henriques <henrix@camandro.org>
|
| |
|
|
|
|
|
| |
The feature was removed from this fork already. This commit brings
the documentation up to date with the source.
|
| |
|
| |
|
|
|
|
|
|
| |
Nowadays, most ignored clients will be bars or other "decorative"
elements. We do not see the use of jumping to those using the window
menu. They also tend to clutter up the menu, so let's ignore them there.
|
|
|
|
|
|
|
|
|
|
| |
If a client is killed and the cursor lands in the root window, no client
is focused. Attempting to cycle back to the last focused client will
instead focus the previous-to-last. Add specific checks that make sure
the cursor is positioned where the user expects it to be. This patch was
taken from a bug report on openbsd-misc [1].
[1] https://marc.info/?l=openbsd-tech&m=155931484124288&w=2
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As cwm was using the screen's default visual and colormap to draw all
client borders, borders for windows that had a depth of 32 bits were not
rendered correctly. The same happened with text in the popup menus which
were recently changed to be drawn in the context of the client.
This commit introduces a Visual reference for each client, and allocates
all potential colors for a client's specific visual and colormap in the
client_ctx struct. These colors are then used to draw client borders and
popup menus.
Additionally, since we touch the drawing code anyway, borders are
reintroduced to the popup menus for better contrast.
|
|
|
|
|
|
| |
These loops never freed cargs and cargs->cmd from bind_ctx. Instead of
adding more calls to free() manually, replace the whole loops with the
conf_unbind_* functions which free the struct properly.
|
|
|
|
|
| |
The previous size calculation did not account for the prompt characters
(0xc2 0xbb and 0xc2 0xab respectively).
|
|
|
|
|
| |
Obscure feature. Confusing if you don't know about it, mostly useless if
you do. Matching on currently visible window titles is enough.
|
|
|
|
|
|
|
|
| |
Whilst arguably useful for certain people, or in rare cases, this
functionality is not needed in the general case.
This will also remove a memory leak where fallback was not properly
freed.
|
|
|
|
|
|
| |
There is no point in keeping this functionality. Saving a few key
strokes is not worth the added complexity of the window manager parsing
the known_hosts file.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There are a couple of issues I found with this functionality:
1) To my knowledge, it is undocumented and extremely specific.
2) Since this function would only complete a full path (by wrapping it
in quotes before passing it on), it is entirely useless when composing a
command that has more than one argument.
3) The snprintf call has no check for truncation, possibly leading to
the path not being quoted properly.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
Original file locations in the OpenBSD tree:
- lib/libc/stdlib/reallocarray.c
- lib/libc/stdlib/strtonum.c
- lib/libc/string/strlcat.c
- lib/libc/string/strlcpy.c
- sys/sys/queue.h
|
|
|
|
|
|
|
| |
This reverts commit 603548105b9bf9ffd11eb053e62db99f9433e821.
Using u_char instead of char will generate signedness warnings for
strtonum(), lookup(), and xstrdup().
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Spawn a window, maximize it in any way, move the cursor to a window border
that is not on the screen's edge and unmaximize again: While the window
goes back the cursor stays at the screen's edge, i.e. focus is lost to the
underlaying window.
Moving, resizing, tiling or snapping windows in any way always moves the
cursor along iff needed, e.g. using MS-[hjkl] to move a small window from
the center to the edge keeps the cursor within window borders -- no matter
what you do with the keyboard, focus stays on that window.
Make CM-f, CM-m, CM-equal and CMS-equal (default bindings) for toggling
full-screen mode, maximization, vertical maximization and horizontal
maximization of the current window drag the cursor along if needed as well.
OK okan kmos dv
|
|
|
|
|
|
|
| |
If a client calls XSetTextProperty for a window to clear all its
properties, then allocated memory within libX11 is not freed.
OK okan@
|
|
|
|
|
|
|
|
|
|
| |
quoted arguments are stored for other window managers.
The quotation handling happens within the while loop without checking if
the "end" limit has been already reached. If this happens, the final
NULL assignment leads to an out of boundary write on stack.
OK okan@
|
|
|
|
| |
htile/vtile actions. From Uwe Werler, with a few manpage tweaks.
|
| |
|
|
|
|
| |
determine what client to set active. Reduces a round trip for every window.
|
| |
|
|
|
|
| |
OK okan@
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
From and OK okan@
Original commit message:
Plug two memory leaks. Also get rid of a variable that is no longer
necessary.
OK okan@
|
| |
|
|
|
|
| |
OK okan@ schwarze@
|
|
|
|
| |
OK okan@
|
| |
|
|
|
|
| |
dosage/removal of wrappers, simplification of name queue, client cycle joins other kb/mb bound functions.
|
|
|
|
| |
Behaviour (or lack there of) noticed by Raf Czlonka.
|
|
|
|
| |
From Artturi Alm (though changed from 'm' to '5')
|