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 absolute monitor workspace selectors #5848

Merged
merged 3 commits into from
May 3, 2024

Conversation

wagyourtail
Copy link
Contributor

@wagyourtail wagyourtail commented May 3, 2024

Describe your PR, what does it fix/add?

This PR adds the ability to select workspaces by monitor using their absolute index. previously, workspaces for a monitor could only be selected in a relative manner ie m-1. this allows absolute using the syntax m~[index].

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

is there a better symbol than ~ to represent absolute?

my usecase:
multimonitor with hyprexpo workspace_method = first r~1 so that it always shows the first workspace on the monitor.

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

I guess. (isn't this what draft pr's are for indicating)

vaxerski
vaxerski previously approved these changes May 3, 2024
Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, wiki mr needed

@vaxerski
Copy link
Member

vaxerski commented May 3, 2024

clang-format oops

wagyourtail added a commit to wagyourtail/hyprland-wiki that referenced this pull request May 3, 2024
@wagyourtail
Copy link
Contributor Author

wagyourtail commented May 3, 2024

wiki mr needed

👍 hyprwm/hyprland-wiki#628

@vaxerski vaxerski merged commit 1d2acbe into hyprwm:main May 3, 2024
10 checks passed
vaxerski pushed a commit to hyprwm/hyprland-wiki that referenced this pull request May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants