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

Feature: option to use desc: Monitor instead of port name #23

Closed
mrwsl opened this issue Jul 6, 2023 · 8 comments · Fixed by #25
Closed

Feature: option to use desc: Monitor instead of port name #23

mrwsl opened this issue Jul 6, 2023 · 8 comments · Fixed by #25
Labels
enhancement New feature or request

Comments

@mrwsl
Copy link

mrwsl commented Jul 6, 2023

Thanks for nwg-displays. This might be a hyprland specific wish, but it would be nice to have the option to use the monitors' description instead of the port name.
The background is that I have multiple workplaces with different monitor setups. Using the description would make using different setups more robust.

@nwg-piotr
Copy link
Owner

Nothing in the Hyprland wiki indicates that we can use generic names, but I've just tried:

monitor=Dell Inc. DELL E2214H 2RK1Y4333FVU,[email protected],1366x0,1.0
monitor=AU Optronics 0x315C,[email protected],0x312,1.0

...and it seems to work well. If so, your request is doable.

@nwg-piotr nwg-piotr added the enhancement New feature or request label Jul 6, 2023
@mrwsl
Copy link
Author

mrwsl commented Jul 6, 2023

It is written above this section (scroll up a bit). It follows the syntax desc: Monitor description.

@nwg-piotr
Copy link
Owner

Thanks.

I'll add what you requested when I'm done with what I'm working on at the moment, probably this week.

@nwg-piotr
Copy link
Owner

nwg-piotr commented Jul 8, 2023

It seem we have a problem. Setting default workspace for a monitor seems not to work with descriptions.

workspace=HDMI-A-1,1

works.

workspace=AOC 2475WR F17H4QA000449,1

does not work.

Also

workspace=1,monitor:HDMI-A-1,default:true

works, but

workspace=1,monitor:AOC 2475WR F17H4QA000449,default:true

does not work.

Unless I'm doing something wrong, we've reached a dead end.

@nwg-piotr
Copy link
Owner

Yes, it does not work yet.

hyprwm/Hyprland#2670

@nwg-piotr
Copy link
Owner

nwg-piotr commented Jul 12, 2023

The problem on the Hyprland side has just been resolved by @MightyPlaza (thanks again!).

@herrwusel - The feature you requested must wait on the desc branch till the nearest Hyprland release. Of course you can test it right now, if you install the development version of Hyprland.

2023-07-12-033412_hypr_screenshot

@mrwsl
Copy link
Author

mrwsl commented Jul 12, 2023

Thanks for your quick implementation. I'll give it a try and report back.

@nwg-piotr
Copy link
Owner

I discovered a minor bug. Will fix later today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants