mirrors/labwc
1
0
Fork 0
mirror of https://github.com/labwc/labwc.git synced 2025-10-29 05:40:24 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

src/output.c: only overwrite the automatic layout if necessary

Browse source 
The wlroots wlr_output_layout provides two different modes of operation:
- automatically
- manually

In automatic mode wlroots reacts to new / removed outputs and resolution
changes and then adjusts the output positions within the layout itself.

The manual mode disables this behavior and thus it is the whole
responsibility of the caller (e.g. us relaying whatever wdisplays /
kanshi provides) to ensure that the layout is somewhat sane. E.g. that
it doesn't have any holes between outputs and there are no overlapping
outputs.

The mode is set for each output individually. To use (and keep using)
the automatic mode, outputs have to be added via the _add_auto() variant
(which we do) and they are not allowed to be moved via _move() (which
we currently do and thus break the automatic layout).

To fix that, this patch compares the user (tool) supplied position to the
automatically calculated position and only if they differ we call _move().
This commit is contained in:
Consolatis 2023-01-29 09:58:42 +01:00 • committed by Johan Malm
parent 6f3043b08d
commit dd1a8f01dd
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
src/output.c
View file

@ -314,8 +314,13 @@ output_config_apply(struct server *server,
}
if (output_enabled) {
wlr_output_layout_move(server->output_layout, o,
head->state.x, head->state.y);
struct wlr_box pos = {0};
wlr_output_layout_get_box(server->output_layout, o, &pos);
if (pos.x != head->state.x || pos.y != head->state.y) {
/* This overrides the automatic layout */
wlr_output_layout_move(server->output_layout, o,
head->state.x, head->state.y);
}
}
if (need_to_remove) {