dreamcat4
commented
1 year ago The behavior is indeed improved in the way as intended (for this PR).
The observed behavior after applying the commit is that:
The clientwin focus (highlight box) can now only land (or be moved on to) those first N 'real' desktops. So regardless of any extra empty grid squares still being rendered... those unused areas are now being correctly ignored / skipped over. And the mouse cannot select them either.
So in terms of what this PR aims to achieve: All confirmed working fine.
However, the "layout" of the desktops is still square. For example, if there are 5 virtual desktops, then it will be arranged like so:
8 8 8 8 8
And an empty row in the bottom to form 3x3 matrix.
Future PR can take config parameter for the dimension of the matrix.
@dreamcat4 please try