Set WM_CLASS and _NET_WM_STATE

Qelxiros / lazybar

A lightweight X11 status bar written in Rust. The spiritual successor to polybar, with some inspiration from cnx.

MIT License

6 stars 2 forks source link

Set WM_CLASS and _NET_WM_STATE #4

Closed cecilialau6776 closed 3 months ago

cecilialau6776 commented 3 months ago

Set WM_CLASS to "lazybar", "Lazybar"
Set _NET_WM_STATE to _NET_WM_STATE_STICKY, _NET_WM_STATE_ABOVE

Qelxiros commented 3 months ago

According to the EWMH spec, _NET_WM_NAME should be used over WM_NAME, and _NET_WM_STATE_ABOVE should not be used by applications. I'm not sure about _NET_WM_STATE_STICKY, but it feels like it should be covered by _NET_WM_WINDOW_TYPE(_DOCK). I'm going to close this for now, but I've opened #5, so I'll come back to this eventually.

cecilialau6776 commented 3 months ago

What about WM_CLASS?

Qelxiros commented 3 months ago

WM_CLASS comes from ICCCM, which lazybar currently doesn't support. I may add it in the future, but I use i3, which doesn't appear to be ICCCM-compliant, which might make testing more difficult.

Qelxiros commented 3 months ago

Turns out I didn't know what ICCCM was. These changes were (approximately) implemented in 14d796d.