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

Replaced always_center_master to slave_count_for_center_master #8871

Merged
merged 2 commits into from
Dec 29, 2024

Conversation

khushal-solves
Copy link
Contributor

@khushal-solves khushal-solves commented Dec 27, 2024

Describe your PR, what does it fix/add?

  • Replaced boolean config parameter always_center_master to an integer value slave_count_for_center_master for more control on when to center master window.
  • Also updated configuration parameter, type, description and related internal variable.

New description: when using orientation=center, make the master window centered only when at least this many slave windows are open. (Set 0 to always_center_master)

This effects center orientation of master layout.
Now, user can control when the master window would be centered which solves #8866
Related Wiki Modification

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?

  • Ready

I hope i've not misunderstood something.
In case you are not interested, feel also free to close this PR :)

Cheers!

- Replaced boolean config parameter `always_center_master` to an integer value `slave_count_for_center_master` for more control on when to center master window.
- Also updated configuration parameter, type, description and related internal variable.

New description: **when using orientation=center, make the master window centered only when at least this many slave windows are open. (Set 0 to always_center_master)**

This effects center orientation of master layout.
Now, user can control when the master window would be centered
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.

wiki mr needed

@khushal-solves
Copy link
Contributor Author

Hi @vaxerski, thank you for approving these changes.
I opened a PR with same name for wiki repo.

Edited description above to include a link to that PR

@vaxerski
Copy link
Member

thanks!

@vaxerski vaxerski merged commit fde569d into hyprwm:main Dec 29, 2024
11 checks passed
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