From d956cb7861b2670aaa553379db099895b045aa5e Mon Sep 17 00:00:00 2001
From: blitzmann <holmes.ryan.90@gmail.com>
Date: Thu, 16 Nov 2017 01:31:28 -0500
Subject: [PATCH] use a different image to denote changed skills

---
 gui/characterEditor.py       |  23 ++++++++++++-----------
 imgs/gui/skill_small_red.png | Bin 0 -> 697 bytes
 2 files changed, 12 insertions(+), 11 deletions(-)
 create mode 100644 imgs/gui/skill_small_red.png

diff --git a/gui/characterEditor.py b/gui/characterEditor.py
index f54a868d4..c5b6b7169 100644
--- a/gui/characterEditor.py
+++ b/gui/characterEditor.py
@@ -316,6 +316,7 @@ class SkillTreeView(wx.Panel):
         self.imageList = wx.ImageList(16, 16)
         tree.SetImageList(self.imageList)
         self.skillBookImageId = self.imageList.Add(BitmapLoader.getBitmap("skill_small", "gui"))
+        self.skillBookDirtyImageId = self.imageList.Add(BitmapLoader.getBitmap("skill_small_red", "gui"))
 
         tree.AppendColumn("Skill")
         tree.AppendColumn("Level")
@@ -423,7 +424,7 @@ class SkillTreeView(wx.Panel):
             level, dirty = sChar.getSkillLevel(char.ID, id)
 
             if dirty:
-                name = "* " + name
+                iconId = self.skillBookDirtyImageId
 
             childId = tree.AppendItem(root, name, iconId, data=('skill', id))
             tree.SetItemText(childId, 1, "Level %d" % int(level) if isinstance(level, float) else level)
@@ -441,14 +442,14 @@ class SkillTreeView(wx.Panel):
             self.btnSecStatus.Enable()
 
         groups = sChar.getSkillGroups()
-        imageId = self.skillBookImageId
         root = self.root
         tree = self.skillTreeListCtrl
         tree.DeleteAllItems()
 
         for id, name in groups:
+            imageId = self.skillBookImageId
             if id in dirtyGroups:
-                name = "* " + name
+                imageId = self.skillBookDirtyImageId
 
             childId = tree.AppendItem(root, name, imageId, data=('group', id))
             tree.AppendItem(childId, "dummy")
@@ -472,7 +473,7 @@ class SkillTreeView(wx.Panel):
                 level, dirty = sChar.getSkillLevel(char.ID, id)
 
                 if dirty:
-                    name = "* " + name
+                    iconId = self.skillBookDirtyImageId
 
                 childId = tree.AppendItem(root, name, iconId, data=('skill', id))
 
@@ -525,9 +526,9 @@ class SkillTreeView(wx.Panel):
             self.skillTreeListCtrl.SetItemText(treeItem,
                                                1,
                                                "Level {}".format(int(lvl)) if not isinstance(lvl, str) else lvl)
-            # @todo: pheonix
-            # if not dirty:
-            #     self.skillTreeListCtrl.SetItemTextColour(treeItem, None)
+
+            if not dirty:
+                self.skillTreeListCtrl.SetItemImage(treeItem, self.skillBookImageId)
 
         while child.IsOk():
             # child = Skill category
@@ -551,10 +552,10 @@ class SkillTreeView(wx.Panel):
         parentID = self.skillTreeListCtrl.GetItemParent(selection)
         parent = self.skillTreeListCtrl.GetItemData(parentID)
 
-        # @todo: pheonix
-        # if parent:
-        #     if parent[1] in dirtyGroups:
-        #         self.skillTreeListCtrl.SetItemTextColour(parentID, None)
+        if parent:
+            if parent[1] in dirtyGroups:
+                self.skillTreeListCtrl.SetItemImage(parentID, self.skillBookImageId)
+
 
         event.Skip()
 
diff --git a/imgs/gui/skill_small_red.png b/imgs/gui/skill_small_red.png
new file mode 100644
index 0000000000000000000000000000000000000000..7a156c781b5d017b03694e3783b800033f4a6ace
GIT binary patch
literal 697
zcmV;q0!ICbP)<h;3K|Lk000e1NJLTq000mG000mO1^@s6AM^iV00006VoOIv0RI60
z0RN!9r;`8x010qNS#tmY3ljhU3ljkVnw%H_000McNliru;R_H37d2tHI1T^+0!B$h
zK~y+T#gX4jQ*ju^Ki{3R-|A*gCp3&9RM1_ZyD=z8gs85f%W!`}*WDXLT?JhT!Jw<W
z5e=^-2orJ%W93}x8i|hD55;-h&e=T2eY<ela?))NyzoBoJ|Etf7yh?It9b^lThXXn
zsw#aWBT9E9BC^xd9r?vY{m$H6;h%u76BAus0|W1zfk4=*RB#1@iaQh%HiyG|kWO2{
z(a|1&&*S3^$71cIpEEPzy<}3;b2*uxpO@PDx~vrn($q9r-`$n*X!J>Y3Y3$kCzT@P
z_dn<ehkNy{EomJb&@fGobREUx5vtF3`QGYkEM=OtUxv{Rcwa1<(MaTmsMW67Y&Nn}
zQ&@!pR;5C{SVVwtVQsDSI+gka;IrGQHyWrd+sM+=(wB>o$U|=^B;xV73}iC&b#~I}
zcH{DRxKyt{2VM@i-R72INcj5;0vOT~iFmbCl8MPlW)lf6IGwl@1+UE}8bYvHt-g7c
z%|1E_DF8M=2!RC0T_MKowOX&`^)k1-OxdzHY&6hJlbm6o9Ug{<eZIFzO)CS(Cp3;p
z0@z>3WR|+q>8t&!D%^@9RUyzVOO%~Xso3q@@9+1xe7?b0F8AXU0B|Iwv<Y$aVPj+E
z#n{+kNeIW|!9ii=^SBKI>2QdGX_DRB%g(N>thL_VDz7TU&D(*%(;>e<_;F{auwAcj
zW_A5LkOY>2Z>@o|02R0bTyOT<KoKYd`+yFZe!HErX`TxZKv%OGKou|mv+3J(768qs
f@FyV8nK{*Ox=Q#f{|=ST00000NkvXXu0mjf7+OE+

literal 0
HcmV?d00001