Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 26, 2024. It is now read-only.

allow configure x and y of outputs#301

Merged
sevz17 merged 1 commit intodjpohly:mainfrom
sevz17:pos-mons
Feb 13, 2023
Merged

allow configure x and y of outputs#301
sevz17 merged 1 commit intodjpohly:mainfrom
sevz17:pos-mons

Conversation

@sevz17
Copy link
Collaborator

@sevz17 sevz17 commented Sep 8, 2022

No description provided.

@fauxmight
Copy link

@sevz17: There exists a branch and a patch that does this and more by @PalanixYT

It allows configuring x and y of outputs (like this PR) and also x-resolution, y-resolution, refresh rate, adaptive sync, and handles custom (non-standard on a given monitor) resolutions.

I merge this branch regularly and have used it daily for many months without any issues.
DOWNSIDE: @PalanixYT 's branch is net +59 lines; yours here is net +6 lines.

@sevz17
Copy link
Collaborator Author

sevz17 commented Sep 27, 2022

@sevz17: There exists a branch and a patch that does this and more by @PalanixYT

It allows configuring x and y of outputs (like this PR) and also x-resolution, y-resolution, refresh rate, adaptive sync, and handles custom (non-standard on a given monitor) resolutions.

I merge this branch regularly and have used it daily for many months without any issues. DOWNSIDE: @PalanixYT 's branch is net +59 lines; yours here is net +6 lines.

@fauxmight, sorry for the late reply; yes I'm aware of that patch, but my intention is not replace that patch just allow configure position just like the readme says

@fauxmight
Copy link

This sounds like a great idea.
This also sounds like something to which there is no current opposition.

Vote +1

@sevz17 sevz17 merged commit 23ede80 into djpohly:main Feb 13, 2023
@sevz17 sevz17 deleted the pos-mons branch February 13, 2023 19:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments