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

show groupings and swallowing in hyprctl #1159

Merged
merged 8 commits into from
Dec 4, 2022
Merged

show groupings and swallowing in hyprctl #1159

merged 8 commits into from
Dec 4, 2022

Conversation

eriedaberrie
Copy link
Contributor

Describe your PR, what does it fix/add?

Shows grouped and swallowed windows in the hyprctl client data.

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

No.

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

Looks good for merging to me, but this is my first time touching a C++ project with more than one source file so let me know if there's anything I did bad I should change.

src/debug/HyprCtl.cpp Outdated Show resolved Hide resolved
src/debug/HyprCtl.cpp Outdated Show resolved Hide resolved
src/debug/HyprCtl.cpp Outdated Show resolved Hide resolved
@eriedaberrie
Copy link
Contributor Author

Do you think grouped should also be returning handles instead of pids?

@vaxerski
Copy link
Member

vaxerski commented Dec 3, 2022

Do you think grouped should also be returning handles instead of pids?

absolutely

@vaxerski
Copy link
Member

vaxerski commented Dec 4, 2022

thanks!

@vaxerski vaxerski merged commit 826e35f into hyprwm:main Dec 4, 2022
fufexan pushed a commit that referenced this pull request Dec 5, 2022
* feat: show groupings and swallowing in hyprctl
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.

None yet

2 participants