added option to choose the default monitor that the cursor will appea…

[ikalco]

Default monitor for cursor #5803

Describe your PR, what does it fix/add?

Adds an option (string) under general:default_cursor_monitor.
Then, whenever a new monitor is added, it checks whether it's name matches the above string.
If it does, then it will set the cursor to the middle of that monitor.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

If the option isn't set then it functions the same as before

Also, technically if a new monitor is added after startup and it matches the name, then the cursor will be set to there.
To fix this you would need to make the code stop checking after startup or after all the initial monitors have been added.
I don't know how you would do that though

Is it ready for merging, or does it need work?

Probably, I tested it on my PC and changing the option does change which monitor the mouse appears in on startup

[vaxerski]

To make this one-time:

• add a static bool once = true inside the if and set it to false once you match one output
• add a static std::chrono::system_clock::time_point firstTimeOrSomethign = std::chrono::system_clock::now(); with the same idea and give an arbitrary max time threshold (10s should be reasonable) after which the thing disables itself

[thejch]

Would it be better to make this part of the monitor rules? it would probably also fix this issue:

Also, technically if a new monitor is added after startup and it matches the name, then the cursor will be set to there.
To fix this you would need to make the code stop checking after startup or after all the initial monitors have been added.
I don't know how you would do that though

[vaxerski]

don't think so, because it can only match one monitor at a time, so there is no benefit to it being a rule.

[ikalco]

I also saw that whenever a new window is created it doesn't follow where the mouse was.
Ex. mouse, by default, on left monitor, window created on right

Is that related?
Maybe the focus needs to be set as well?

[vaxerski]

as it's getting quite large, I'd move it to a separate function e.g. static void checkDefaultCursorWarp or something

[vaxerski]

I also saw that whenever a new window is created it doesn't follow where the mouse was.

you should also g_pCompositor->setactivemonitor and g_pInputManager->refocus() for that

[vaxerski]

lgtm, wiki mr needed

[ikalco]

Does this work?
hyprwm/hyprland-wiki#627

[vaxerski]

thanks!