search

Cloudef / wlc

High-level Wayland compositor library
MIT License
331 stars 58 forks source link

Use WM_CLASS class instead of instance (fix #218) #222

Closed ljrk0 closed 7 years ago

ljrk0 commented 7 years ago

Usually the second part of WM_CLASS (the actual window class, not the instance) is expected (compare to i3wm class property) so we use this one.

This fixes #218

ljrk0 commented 7 years ago

Finally got around fixing this, what do you think?

ljrk0 commented 7 years ago

Done squashing

ddevault commented 7 years ago

Would be nice to also be able to get the instance out of wlc.

ljrk0 commented 7 years ago

This'd be the old functionality of the function, we'd just to add a new one, not difficult. The only question is whether to add it in here or regard it as a new feature and seperate it.

Cloudef commented 7 years ago

I'll merge this now. Just do new PR if you decide to add instance.