Skip to content

Conversation

@LlinksRechts
Copy link
Contributor

This repository does not seem to be maintained any more. However, since I have been maintaining my own version of snaptile, I figured I'd open a PR for all my changes, in case anyone stumbles upon this and wants to use the fixes + features I have implemented, even if this PR is never merged.
After this short disclaimer, now for the content of this PR:

  • Fixed multiscreen mode (see Bugs in dual screen mode #19, Fixes for dual screen mode #20)
  • Respond to SIGTERM, terminate application (-> to allow kill <pid>)
  • Replaced deprecated GObject calls with newer methods
  • Fixed a bug related to Xlib which ate key release events, leading to wrong placement after multiple tiling events
  • Add fill mode (explained below)
  • Prevent focus stealing for mouse hover activated windows (previously, if two tiles were selected, the window was moved after the first tile, and the window behind that one might have been activated if focus is mouse hover controlled)
  • Code restructuring

Fill mode

This mode can be enabled with the -f switch; if enabled, double pressing the key for a single tile greedily fills as many tiles around this one as possible without overlapping other windows. Windows already intersecting the tile itself are ignored. For example, in the grid

1 2 3 4
5 6 7 8
9 10 11 12

if 6 contains another window and 9 is pressed twice, the window will span 9 to 12. If 11 also contains another window, however, 1 to 9 will be chosen (since that spans 3 tiles, as opposed to only 2 for 9 to 10).

Lukas Rysavy and others added 11 commits May 27, 2020 08:46
to be able to handle differently sized monitors
with call to newer GLib method
...which ate key release events (apparently, this is intentional), see
https://bugs.freedesktop.org/show_bug.cgi?id=99280
enable with the -f switch

on double press on a single tile, greedily fills as many tiles
as possible around this one without intersecting with other windows.
windows which intersect with the given tile are excluded (or no
results would be possible)
@boussou
Copy link

boussou commented Dec 8, 2020

Thanks !
awesome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants