From fd3bd7c6e0433b753db94b79f986da1bda30f343 Mon Sep 17 00:00:00 2001
From: WrinklyNinja <wrinklyninja1@gmail.com>
Date: Mon, 13 May 2013 18:37:18 +0100
Subject: [PATCH] Added icon to report viewer, fixed filter-related oddness.

---
 README.md                   | 3 +++
 examples/resource/script.js | 4 ++--
 src/generators.h            | 6 ++++--
 src/gui/viewer.cpp          | 1 +
 4 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index 3ebb1f23..09b6555f 100644
--- a/README.md
+++ b/README.md
@@ -58,6 +58,7 @@ I'm considering using glog to handle logging, should really get that implemented
 - [x] Add a setting that lets users choose whether to use the viewer window or their own browser when viewing BOSS's report.
 - [x] Somehow implement filter settings memory for the BOSS report.
 - [ ] Implement checking of details part of report against previous report.
+- [x] Either remove the Bash Tag suggestions filter or implement their display in the report as a non-note message type.
 
 
 ## Optimisation
@@ -77,6 +78,7 @@ LOOT will be a self-contained installation that can be dropped anywhere. It will
 
 Directory structure will be:
 
+```
 /
     BOSS.exe
     resource/
@@ -95,6 +97,7 @@ Directory structure will be:
         userlist.yaml
         report.html
     ...other game folders with same structure as Oblivion...
+```
 
 
 ## Required Libraries
diff --git a/examples/resource/script.js b/examples/resource/script.js
index 1769cbca..c1a0a9b0 100755
--- a/examples/resource/script.js
+++ b/examples/resource/script.js
@@ -109,11 +109,11 @@ function toggleMessages(evt) {
             var filterMatch = false;
             if (evt.target.id == 'hideAllPluginMessages') {
                 filterMatch = true;
-            } else if (evt.target.id == 'hideNotes' && listItems[i].className.indexOf('note') != -1) {
+            } else if (evt.target.id == 'hideNotes' && listItems[i].className.indexOf('say') != -1) {
                 filterMatch = true;
             } else if (evt.target.id == 'hideBashTags' && listItems[i].className.indexOf('tag') != -1) {
                 filterMatch = true;
-            } else if (evt.target.id == 'hideDoNotCleanMessages' && listItems[i].className.indexOf('dirty') != -1 && (listItems[i].textContent.indexOf('Do not clean.') != -1 || listItems[i].textContent.indexOf('Do not clean.') != -1)) {
+            } else if (evt.target.id == 'hideDoNotCleanMessages' && listItems[i].textContent.indexOf('Do not clean.') != -1) {
                 filterMatch = true;
             }
             if (filterMatch) {
diff --git a/src/generators.h b/src/generators.h
index 4531b0a4..90f23c09 100644
--- a/src/generators.h
+++ b/src/generators.h
@@ -38,7 +38,9 @@ namespace boss {
 
         if (type == "say")
             content = "Note: " + content;
-        else if (type == "warn")
+        else if (type == "tag") {
+            content = "Bash Tag Suggestion(s): " + content;
+        } else if (type == "warn")
             content = "Warning: " + content;
         else
             content = "Error: " + content;
@@ -324,7 +326,7 @@ namespace boss {
                         content += "Bash Tags suggested for addition are " + add.substr(2) + ". ";
                     if (!remove.empty())
                         content += "Bash Tags suggested for removal are " + remove.substr(2) + ". ";
-                    messages.push_back(Message("say", content));
+                    messages.push_back(Message("tag", content));  //Special type just for tag suggestions.
                 }
 
                 AppendMessages(plugin, messages, warnNo, errorNo);
diff --git a/src/gui/viewer.cpp b/src/gui/viewer.cpp
index 79fbb711..91e9925b 100644
--- a/src/gui/viewer.cpp
+++ b/src/gui/viewer.cpp
@@ -40,4 +40,5 @@ Viewer::Viewer(wxWindow *parent, const wxString& title, const boss::Game& game)
 
     SetSizer(topsizer);
     SetSize(800,600);
+    SetIcon(wxIconLocation("BOSS.exe"));
 }