-
-
Notifications
You must be signed in to change notification settings - Fork 278
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Auto-format OnScreen
and ScreenCorners
to HLS
#918
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you planning on working on these modules in the future? If not, I don't personally think that putting another indirection into the git history is worth it, especially since the other contrib modules don't follow a certain style, but are sort of all over the place
@slotThe Yes, there might be an issue with screen corners on multi monitor setups. Working on it will be easier if HLS does not change the entire file once I save it... |
There are definitely issues with screen corners on multi-monitor setups:
Note that the first one actually requires some significant design work to fix. I made a start on it, but had to put it on the back burner. |
@geekosaur Yes, exactly. Do you have an issue for it or should we create one? Can you also link your previous work? |
I chose not to file an issue yet after seeing how much work the redesign would be; and unfortunately I'm not finding the initial (very preliminary) design work, although I remember the fundamentals. The gist is that, especially with window sides support, you need to track sides per monitor and place the corners according to those.
Both the lower left and right borders and corners are unreachable currently, because the root window's bounding box is dominated by the center monitor. (See the asterisks at the bottom; those are the current bottom corners.) Some additional questions need to be answered, which is another reason I haven't put anything up:
|
BTW, there is a pretty strong argument against trying to use the edges of the center window: X11 doesn't quite handle the difference between size and resolution well. The above is not quite how my current screen layout works (the left and right monitors are actually the same size and resolution in my setup), but the mouse often gets "stuck" moving into the center monitor (my laptop's display) from either, possibly because it's physically smaller while having a larger resolution so something thinks it's moving into an unreachable region. |
Created an issue: #919 Please merge the PR so we can have clean diffs in the future. And let's have the discussion on the issue... |
7f46152
to
7f0f0ad
Compare
Description
Auto-format both my modules
OnScreen
andScreenCorners
with HLS. No other (functional) changes were done.Checklist
I've read CONTRIBUTING.md
I've considered how to best test these changes (property, unit,
manually, ...) and concluded: XXX
I updated the
CHANGES.md
file