Closed felixfung closed 1 year ago
i am going to merge this, then test this pr today. since it is easy one to test. is not much burden here.
but i am not going to be testing you other future prs, until after a future time that i have then worked on and fixed for the multi monitor support (the priority task)
however (for the next times) i can still do some lighter code review, before merging
i hope this clarifies here. because it's adhoc / not so formal working arrangements. to keep both on same page
(code was rebased)
testing result:
Previous behaviour (before merge):
____________________
[_____][_____][_____]
[_____][_____]
____________________
The new observed behaviour:
____________________
[_____][_____][_____]
[_____][_____]
____________________
So I don't see any issue or unindended consequences here with this. All seems fine.
This is continuation of #157. @dreamcat4 please test
the paging layout is rectangular such that screenwidth == ceil(sqrt(screencount)) and the screenheight == ceil(screencount / screenwidth)