updated progress; netwmicon patch cancelled

1 files changed, 29 insertions, 2 deletions

diff --git a/todo.org b/todo.org
index 19ec13d..018cc92 100644
--- a/todo.org
+++ b/todo.org
@@ -10,11 +10,38 @@ ST Patches
 - [ ] run terminal tests in tmux
 - [ ] lf + ueberzug + image previews no tmux
 - [ ] lf + ueberzug + image previews in tmux
-** TODO [#C] netwmicon
+* ST Patches Aborted
+** CANCELLED [#C] netwmicon
 *** Verification Steps
 - run xprop and select open st window
+*** 2023-07-25 @ 09:27:19 -0500 patch failed even after manual intervention; cancelling
+rejected lines in both config.mk and Makefile corrected
+however, xprop didn't show the icon no matter where I set ICONPREFIX (see description below)
 *** Description and URLs
-* ST Patches Aborted
+Description
+
+Enables to set _NET_WM_ICON with a png-image.
+
+Generally the icon of an application is defined by its desktop-entry. The patch desktopentry serves this purpose. Unfortunately, some programs like tint2 or alttab can't make use of the desktop-entry and rely instead on a hardcoded icon which has to be defined by the application itself with the window-propery _NET_WM_ICON. Since st doesn't define _NET_WM_ICON this programs can't display the correct icon for st even if a desktop-entry exists. This patch solves this problem.
+Dependencies
+
+    gd
+
+Defining an icon
+
+By default each time st starts it will search for a file with the name st.png under /usr/local/share/pixmaps/. If you put an image with this name in the root-directory of the st-repository and call make install the image will be installed in /usr/local/share/pixmaps/ automatically. Otherwise you have to put the file there manually. You can try it out with this icon st.png (credit: flat-remix).
+Using with desktopentry
+
+If you use the desktopentry-patch you should adjust the icon in the file st.desktop by replacing the line Icon=utilities-terminal with Icon=st.
+
+Most programs which use the desktop-file to determine the icon should be able to find st.png under /usr/local/share/pixmaps/. If not report it to me and try changing in confing.mk the line ICONPREFIX = $(PREFIX)/share/pixmaps to ICONPREFIX = $(PREFIX)/share/icons/hicolor/256x256/apps/ and install the icon there.
+Call to action
+
+Maybe somebody would like to create an icon with the st-logo so we could add it here instead of using the icon from flat-remix.
+
+https://st.suckless.org/patches/netwmicon/
+https://st.suckless.org/patches/netwmicon/st-netwmicon-0.8.5-v2.diff
+
 ** CANCELLED [#C] colorschemes
 *** 2023-07-23 @ 17:34:43 -0500 No need with xresources
 Not needed now that I can configure st attributes via Xresources