Added WM_NAME as _NET_WM_NAME fallback (older apps)

Separated Props and Structs as submodules
Improved performance while maintaining multi-display, multi-screen and multi-root support
Added typing_extensions dependency at runtime

Author: Kalmat

README.md

@@ -1,5 +1,6 @@
 # EWMH-lib
 [![Type Checking](https://github.com/Kalmat/EWMHlib/actions/workflows/type-checking.yml/badge.svg)](https://github.com/Kalmat/EWMHlib/actions/workflows/type-checking.yml)
+[![PyPI version](https://badge.fury.io/py/ewmhlib.svg)](https://badge.fury.io/py/ewmhlib)
 
 Extended Window Manager Hints implementation in Python 3 and python-xlib, which allows to easily query and control 
 Window Managers following these standards.
@@ -49,7 +50,7 @@ Class to access application window features.
 
 To instantiate this class only a window id is required. It is possible to retrieve this value in several ways:
 
-- Target a specific window using an external module (e.g. `PyWinCtl.getAllWindowsWithTitle(title)`)
+- Target a specific window using an external module (e.g. `pywinctl.getAllWindowsWithTitle(title)` or `pywinctl.getActiveWindow()`)
 - Retrieve it from your own application (e.g. PyQt's `winId()` or TKinter's `frame()`)
 
 Note that, although a root is also a window, most of these methods will not likely work with it.
@@ -173,9 +174,9 @@ or
 
     python3 -m pip install ewmhlib
 
-Alternatively, you can download the wheel file (.whl) available in the [Download page](https://pypi.org/project/EWMHlib/#files) and the [dist folder](https://github.com/Kalmat/EWMHlib/tree/master/dist), and run this (don't forget to replace 'x.x.xx' with proper version number):
+Alternatively, you can download the wheel file (.whl) available in the [Download page](https://pypi.org/project/EWMHlib/#files) and the [dist folder](https://github.com/Kalmat/EWMHlib/tree/master/dist), and run this (don't forget to replace 'x.xx' with proper version number):
 
-    pip install EWMHlib-x.x.xx-py3-none-any.whl
+    pip install EWMHlib-x.xx-py3-none-any.whl
 
 You may want to add `--force-reinstall` option to be sure you are installing the right dependencies version.
 
@@ -250,6 +251,9 @@ An (likely) incomplete list of EWMH-compliant window managers is (via Wikipedia,
 |        xmonad        |     (4)     |
 
 (1) Through 0.65 / from 0.70
+
 (2) As of 4.0.0
+
 (3) Releases following and including version 1.1.0 follow the EWMH standard
+
 (4) Must activate EWMH (XMonad.Hooks.EwmhDesktops)

src/ewmhlib/_ewmhlib.py

@@ -9,7 +9,7 @@
 import os
 import threading
 import time
-from typing import Optional, cast, Callable, Union, List, Tuple, Iterable
+from typing import Optional, cast, Callable, Union, List, Tuple
 
 import Xlib.display
 import Xlib.protocol
@@ -258,28 +258,37 @@ def getPropertyValue(prop: Optional[Xlib.protocol.request.GetProperty], text: bo
     Extract data from retrieved window/root property
 
     :param prop: Xlib.protocol.request.GetProperty struct from which extract data
-    :param text: set to ''True'' to convert the atoms (int) to their names (str)
+    :param text: set to ''True'' to convert the atoms (int) to their names (str), if possible
     :param display: display to which window belongs to (defaults to default display)
     :return: extracted property data (as a list of integers or strings) or None
     """
     if prop and hasattr(prop, "value"):
         # Value is either str, bytes (separated by '\x00' when multiple values) or array.array of integers.
         # The type of array values is stored in array.typecode ('I' in this case).
-        valueData: Union[array.array[int], bytes] = prop.value
-        if isinstance(valueData, str) or isinstance(valueData, int):
+        valueData: Union[str, int, bytes, array.array[int], List[str], List[int]] = prop.value
+        if isinstance(valueData, str):
             return [valueData]
-        if isinstance(valueData, bytes):
+        elif isinstance(valueData, int):
+            result = valueData
+            if text and valueData != 0:
+                try:
+                    result = display.get_atom_name(valueData)
+                except:
+                    pass
+            return [result]
+        elif isinstance(valueData, bytes):
             resultStr: List[str] = [a for a in valueData.decode().split("\x00") if a]
             return resultStr
-        elif isinstance(valueData, array.array):
+        elif isinstance(valueData, array.array) or isinstance(valueData, list):
             if text:
-                resultStr = [display.get_atom_name(a) for a in valueData if isinstance(a, int) and a != 0]
-                return resultStr
-            else:
+                try:
+                    resultStr = [display.get_atom_name(a) for a in valueData if isinstance(a, int) and a != 0]
+                    return resultStr
+                except:
+                    text = False
+            if not text:
                 resultInt: List[int] = [a for a in valueData if isinstance(a, int)]
                 return resultInt
-        # Leaving this to detect if data has an unexpected type
-        return [a for a in valueData] if isinstance(valueData, Iterable) else [valueData]
     return None