Fix X11 and Wayland - Images opened in unfocussed Tmux window overlaid on top of active window/session

[twidxuga]

Fixes issue 213, on both Wayland and X11.

Note that for X11, a small change to add_image in x11.cpp was still required, to ensure no flickering (even if just for an instant) in the active session/window when initializing a new image in an unfocussed session/window. I tested in Wayland, and no change of that module was required in that case.

• Fix starting ueberzugpp in an unfocussed window or session results in image being generated on top of the focussed window and session, instead of hidden in the correct ones, and also in the wrong position.

Additionally:

• Removes unused include of the utilities library, introduced in PR #212

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[twidxuga]

I found that you are also working on ueberzugpp-new, likely a refactor of this branch.Since tmux integration is likely to also be relevant in the new project, my two cents are to make the initial image visibility state independent of the image display method (wayland, X11, etc.). In other words, don't let equivalent to X11 or Wayland modules show or hide images after initialization. Simply delegate showing/hiding the image state to the app, so that it can decide when to show or hide images independently.

[jstkdng]

yeah, it's a refactor of the whole project, a lot of technical debt here.

Simply delegate showing/hiding the image state to the app, so that it can decide when to show or hide images independently.

Most applications that use u++ just send either add or remove commands, they just want auto preview managing. I do wonder why old ueberzug doesn't have this issues.

[jstkdng]

I just tried this changes and it broke another functionality that works with regular ueberzug. When you have two tmux clients connected to the same session and you open ueberzugpp no window shows up.

[twidxuga]

Thanks for checking. I can have a look, but likely only next week, since I'm travelling atm.