Bug #7325

Fix application icons in "task list" in Windows 8 camouflage

Added by anonym 2014-05-28 22:13:27 . Updated 2014-07-22 23:11:31 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Camouflage
Target version:
Start date:
2014-05-28
Due date:
% Done:

100%

Feature Branch:
bugfix/7325-task-list-icons
Type of work:
Code
Blueprint:

Starter:
0
Affected tool:
Deliverable for:

Description

With “task list” I mean either window-picker-applet, or the good old “Window List”. It’s enough to fix this for the application we currently have launcher for.


Subtasks


History

#1 Updated by alant 2014-06-05 03:08:50

pidgin : use of /usr/share/icons/hicolor//pidgin.png is hardcoded so the theme is not applied. We should install icons there.

#2 Updated by alant 2014-06-05 03:31:56

claws : icon is bundeled into the binary (see src/stock_pixmap.c in claws-mail source) but there seems to be support of theming.

iceweasel : icon seems to be a link in /usr/share/pixmaps/iceweasel.png. Changing this link actually changes the icon in task list.

#3 Updated by alant 2014-06-05 05:33:28

claws: creating a minimal theme containing only claws_mail_icon.xmp and .claws_themeinfo and setting pixmap_theme_path in the [Common] section of .claws-mail/clawsrc works to change system tray icon.

#4 Updated by alant 2014-06-05 06:22:07

claws: the following script works to change the theme but if one has persistence enabled, this change will be made persistent. However, without the theme installed, this has no effect.

CONFSECTION="\[Common\]"
CONFLABEL="pixmap_theme_path"
CONFVALUE="/home/amnesia/.claws-mail/themes/Windows8"

if grep "$CONFLABEL" "$HOME/.claws-mail/clawsrc"; then
    sed -e "s;$CONFLABEL=.*$;$CONFLABEL=$CONFVALUE;" -i "$HOME/.claws-mail/clawsrc"
else
    sed -e "/$CONFSECTION/ a $CONFLABEL=$CONFVALUE" -i "$HOME/.claws-mail/clawsrc"
fi

#5 Updated by alant 2014-06-05 08:59:28

  • % Done changed from 0 to 30
  • Feature Branch set to bugfix/7325-task-list-icons

This should be fixed in tails:bugfix/7325-task-list-icons but one should currently execute tails-win8-theme-adapt-icons as root manually to test it.

#6 Updated by alant 2014-06-05 09:36:33

  • Status changed from Confirmed to In Progress

#7 Updated by alant 2014-06-06 06:15:51

  • % Done changed from 30 to 40

> one should currently execute tails-win8-theme-adapt-icons as root manually to test it.

This should be fixed with the last version of the branch combined to tails-greeter’s branch bugfix/7325-task-list-icons. To be tested.

#8 Updated by alant 2014-06-07 08:54:15

  • % Done changed from 40 to 60
  • QA Check changed from Dev Needed to Ready for QA

Test results are positive.

#9 Updated by alant 2014-06-10 05:01:53

  • Assignee changed from alant to anonym

#10 Updated by anonym 2014-06-11 16:40:46

  • Category set to Camouflage

#11 Updated by anonym 2014-06-19 18:13:49

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • % Done changed from 60 to 100
  • QA Check changed from Ready for QA to Pass

#12 Updated by BitingBird 2014-07-22 23:11:31

  • Status changed from Fix committed to Resolved