Skip to content

Conversation

@BlackCherryCat
Copy link
Contributor

Added Niri configuration example

Added Niri configuration example
Improved exit handling of the Niri config
Copy link
Owner

@rharish101 rharish101 left a comment

Choose a reason for hiding this comment

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

I tried it and it works well. Thanks for the PR.

README.md Outdated
Comment on lines 189 to 204
output "DP-1" {
// Scale is a floating-point number, but at the moment only integer values work.
scale 1
// Transform allows to rotate the output counter-clockwise, valid values are:
// normal, 90, 180, 270, flipped, flipped-90, flipped-180 and flipped-270.
transform "normal"
// Resolution and, optionally, refresh rate of the output.
// The format is "<width>x<height>" or "<width>x<height>@<refresh rate>".
// If the refresh rate is omitted, niri will pick the highest refresh rate
// for the resolution.
// If the mode is omitted altogether or is invalid, niri will pick one automatically.
// Run `niri msg outputs` while inside a niri instance to list all outputs and their modes.
mode "[email protected]"
}
Copy link
Owner

Choose a reason for hiding this comment

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

Is this part required for Niri in multi-monitor setups? I tried Niri without this section on a single monitor setup, and it worked fine. I'd like to keep the example config as simple as possible.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's not necessary but you will have to keep in mind that it will always gonna pick the highest resolution at 60 hz for all monitors, that's why I included it, but as long as the user understands Niri config they will figure it out, so it's fine for me to remove that part

Remove unnecessary part
@rharish101 rharish101 merged commit a011d5d into rharish101:main Sep 28, 2025
1 check passed
@rharish101
Copy link
Owner

Thanks a lot for your PR!

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