KittyKatt / screenFetch

Fetches system/theme information in terminal for Linux desktop screenshots.
GNU General Public License v3.0
3.84k stars 454 forks source link

EWMH Hints should have priority #187

Open Airblader opened 9 years ago

Airblader commented 9 years ago

I think that for EWMH-compliant window managers the _NET_WM_NAME should be given priority. I have a fork of i3 and would like screenfetch to recognize it under its name, but since i3 is detected first, the EWMH hint will not be considered.

This also makes sense in general, though. If the wm offers its name, it should be taken. Looking up processes and mapping that (or so I understand it works right now) by going through a hardcoded list should be the fallback.

KittyKatt commented 9 years ago

I definitely agree. That's a part of the code I should revisit soon and I'll definitely find a way to do this. :)