[FEAT] Plugin: monitor - Support better matching for description field

[dantefromhell]

[FEAT] Plugin: monitor - Support better matching for description field

[fdev31]

ok then you can ignore your remark about the "startswith".

Is "Samsung".topOf = "BOE" working ?
Can you provide the logs ?

[dantefromhell]

Not sure if this is related, but through some testing I came to the conclusion that the following config does not work

[monitors.placement]
"Samsung Electric Company LU28R55 HNMNB00590".top_Center_Of = "(eDP-1)"

It does not match the monitor connected to eDP-1.

While the following config works

[monitors.placement]
"Samsung Electric Company LU28R55 HNMNB00590".top_Center_Of = "BOE 0x095F"

I'm wondering if this related to hyprland not showing the port name in the description field of hyprctl monitors ?

My current output is

❯ hyprctl version
Hyprland, built from branch  at commit 1c460e98f870676b15871fe4e5bfeb1a32a3d6d8  ().
Date: 2024-02-28
Tag:

flags: (if any)

❯ hyprctl monitors
Monitor eDP-1 (ID 0):
	[email protected] at 2560x0
	description: BOE 0x095F
	make: BOE
	model: 0x095F
	serial:
	active workspace: 2 (2)
	special workspace: 0 ()
	reserved: 0 30 0 0
	scale: 1.00
	transform: 0
	focused: no
	dpmsStatus: 1
	vrr: 0
	activelyTearing: false

Monitor DP-4 (ID 1):
	[email protected] at 0x0
	description: Samsung Electric Company LU28R55 HNMNB00590
	make: Samsung Electric Company
	model: LU28R55
	serial: HNMNB00590
	active workspace: 4 (4)
	special workspace: 0 ()
	reserved: 0 30 0 0
	scale: 1.50
	transform: 0
	focused: yes
	dpmsStatus: 1
	vrr: 0
	activelyTearing: false

[fdev31]

This is already implemented, maybe you are using "old" behavior by forcing full_relayout=false ?

if not, then it is implemented as you suggest ("contains", not "regex", I want to keep it simple for the users...)

On the other hand, I'm wondering why you don't have the port name in the description... indeed this prevents from using the "port" syntax... I'll try to figure a workaround for that issue.

[dantefromhell]

This is already implemented, maybe you are using "old" behavior by forcing full_relayout=false ?

I'm using commit a85401c and full_relayout is set to default value.

[fdev31]

Closing since this requests an existing feature, assuming this is duplicate of #64

[dantefromhell]

Closing since this requests an existing feature, assuming this is duplicate of #64

Agreed.