mirrors/labwc
1
0
Fork 0
mirror of https://github.com/labwc/labwc.git synced 2026-02-06 04:06:33 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

desktop: Cycle first to topmost view if not already focused

Browse source 
The topmost view may not always be the focused view; for example,
when Audacious's main window is focused but the floating Search
Tool window remains on top of it.  In that case the floating window
(not the main window) should be the first view selected in the
window switcher.
This commit is contained in:
John Lindgren 2022-09-02 16:41:27 -04:00 • committed by Consolatis
parent 2e81cc13d4
commit 79d9cfcda6
1 changed files with 31 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

35
src/desktop.c
View file

@ -188,11 +188,38 @@ struct view *
desktop_cycle_view(struct server *server, struct view *start_view,
enum lab_cycle_dir dir)
{
struct view *view = start_view ? start_view : first_view(server);
if (!view) {
return NULL;
/*
* Views are listed in stacking order, topmost first. Usually
* the topmost view is already focused, so we pre-select the
* view second from the top:
*
* View #1 (on top, currently focused)
* View #2 (pre-selected)
* View #3
* ...
*
* This assumption doesn't always hold with XWayland views,
* where a main application window may be focused but an
* focusable sub-view (e.g. an about dialog) may still be on
* top of it. In that case, we pre-select the sub-view:
*
* Sub-view of #1 (on top, pre-selected)
* Main view #1 (currently focused)
* Main view #2
* ...
*
* The general rule is:
*
* - Pre-select the top view if NOT already focused
* - Otherwise select the view second from the top
*/
if (!start_view) {
start_view = first_view(server);
if (!start_view || start_view != desktop_focused_view(server)) {
return start_view; /* may be NULL */
}
}
start_view = view;
struct view *view = start_view;
struct wlr_scene_node *node = &view->scene_tree->node;
assert(node->parent);