Skip to content
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

Add movecursortomonitor dispatcher #8998

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Insprill
Copy link

@Insprill Insprill commented Jan 9, 2025

Describe your PR, what does it fix/add?

Adds the movecursortomonitor dispatcher to move the cursor to a specific monitor.

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

I think I did the hyprctl completions correctly, but please double-check.

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

Yes

@Insprill Insprill force-pushed the feature/movecursortomonitor branch from edc86fc to 18e2125 Compare January 9, 2025 02:49
@vaxerski
Copy link
Member

vaxerski commented Jan 9, 2025

...what's the point?

@Insprill
Copy link
Author

Insprill commented Jan 10, 2025

My main use case is part of a workaround for the split-monitor-workspaces plugin I use. I have the cursor:default_monitor config value set, but when the plugin loads, it moves the default workspace (and cursor) to a different monitor, making that config option useless. With this MR, I used a hyprctl command to set the cursor back to my default display after loading plugins.

Since there are already dispatchers for moving the cursor to an arbitrary position and the corner of a window, as well as for moving workspaces to specific monitors, I thought adding one to move the cursor to a monitor was fitting. If it's unwanted, I'll fix it in the plugin itself (which I'll probably do anyways), but I figured I'd try to upstream something first in case it'd help others, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants