Skip to content

Comments

Support for tiling over multiple monitors#5

Open
sepich wants to merge 10 commits intodprokscha:masterfrom
sepich:master
Open

Support for tiling over multiple monitors#5
sepich wants to merge 10 commits intodprokscha:masterfrom
sepich:master

Conversation

@sepich
Copy link

@sepich sepich commented Apr 5, 2015

No description provided.

@dprokscha
Copy link
Owner

Hi @sepich , I really like that feature of tiling clusters over multiple monitors. Maybe I am a bit pedantic, but in its current state, I wont merge this feature, because I have some special acceptance criterias:

  • Do not use the numpad keys as shortcuts. Not every keyboard has a numpad (e.g. laptops).
  • Do not limit the number of monitors or allow a higher number of monitors. I know some people using more than 3 monitors.
  • You could use the function keys instead.
  • Do not ignore the cluster groups - your code tiles all windows of all cluster groups. But what, if I only want to tile the windows of a specific cluster group? To enhance the usability it would be awesome, if I could tile a specific cluster group to e.g. monitor 3 and another cluster group to monitor 2.

Just my 2 cents 😈

@dprokscha dprokscha self-assigned this Apr 8, 2015
@sepich
Copy link
Author

sepich commented Apr 19, 2015

It is easy to tile only active cluster, but it will overlap with windows from others clusters. So, when you typing, you don't know if you type in other windows or not. That was my usual worry when using multiple clusters and i thought that it's more useful to tile to see all the windows and then type only in some of them.
Then i have to turn off named clusters functionality at all (commented out all the Ctrl+Alt+[0-9]) as it intersects with other app's hotkeys.
Also, i have no >3 monitors configuration to test at, but code should take any number of monitors.
Good question about which hotkeys to use, will wait until your implementation ;) But for now I already need this tiling, that's why I have to quick fix my own version.
So, you can just consider this as possible feature request with some code example ;)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants