master layout - allow usage of layoutmsg swapwithmaster to optionally disable swapping master with a slave

[icejix]

I am writing this issue because I am attempting to use hyprland on my mobile phone. I would like a layout in which you can touch one of the unfocused slave windows in order to focus it, and set it as the master. However, I do not want to touch the master window, and then have it become a slave again.

I am using the plugin hyprgrass in order to bind touch gestures (in this case, a single tap anywhere on the screen) to execute dispatchers (in this case, layoutmsg swapwithmaster master). This feature should be implemented into hyprland's master layout as an extra layoutmsg command, or an option for layoutmsg swapwithmaster.

Expected behaviour for my specific problem, but not compatible with other setups:

• Add an option to master to allow swapping any touchscreen tapped slave window with the master window.

Expected behaviour, more compatible with other setups, my preferred method:

• Add an option to layoutmsg to allow swapping any child into the master position, but when executed with focused master, effectively does nothing.

This could possibly be implemented in a plugin or script, but I'm not experienced with those. I think editing the source code of Hyprland's master layout will be easier for my problem.

[icejix]

I would also like to point out that, after downloading Hyprland's source code, I found the following lines:

https://github.com/hyprwm/Hyprland/blob/af2fdb5d58f5d3017aafd78a8ddafb40f710e34b/src/layout/MasterLayout.cpp#L1109C13-L1118C14

Recompiling after commenting these lines out completely solved my issue. Of course, I have not added an option, but have explicitly removed the lines that were causing my specific problem. Solving this the "correct way" would seem to be extremely easy for anyone willing to add an extra option to this layoutmsg, but I don't know C++, so I cannot fix it myself.

[icejix]

I was told by the wiki to post an additional comment with more info if the discussion is stale. I don't have any additional information to add, other than that I'm still recompiling Hyprland from source. Can someone look into this?

[vaxerski]

Add an option to layoutmsg to allow swapping any child into the master position, but when executed with focused master, effectively does nothing.

this is something I could add

[icejix]

Thank you! I am very glad to see that I won't need to be recompiling anything anymore (once the latest version is published, of course). Have a wonderful day.

[vaxerski]

#11042