[hyprlock] Add more description for monitor field #746
Conversation
Change-Id: I95986cdf8f5bbfd45a41642849c66d6aefb23dfe
|
@PaideiaDilemma is this correct? |
| {{< callout >}} | ||
|
|
||
| Pay attention! You do not need to add `desc:` prefix like in | ||
| [`hyprland.conf`](../Configuring/Monitors.md) | ||
|
|
||
| {{</ callout >}} |
There was a problem hiding this comment.
That is not true.
The current check is surf->output->stringDesc.starts_with(c.monitor)
There was a problem hiding this comment.
Should we support prefixing with desc: as well for consistency?
There was a problem hiding this comment.
The desc: variant stops work for me at all. The ripgrep did not find any occurrences of desc: in sources. I found the line @PaideiaDilemma talking about in sources and tries to remove the desc: from config. And it worked.
So I decided to update doc and mention this peculiarity.
I think, we should to add prefix for consistency too, but we should have clear actual documentation as well
There was a problem hiding this comment.
I create a MR in hyprlock to support desc:.
Sorry I initially misunderstood the wording, because i read it too fast. You were of course correct on this :)
There was a problem hiding this comment.
I think you can remove the this section, but the rest is fine.
The
monitorsfield of hyprlock was not described clearly. Trying to fix that.