Move simple colors to themes.

Need add to all themes, or all simple colors will be black.

2 files changed, 29 insertions, 28 deletions

diff --git a/src/gui/widgets/browserbox.cpp b/src/gui/widgets/browserbox.cpp
index bb83c1159..65a25363d 100644
--- a/src/gui/widgets/browserbox.cpp
+++ b/src/gui/widgets/browserbox.cpp
@@ -68,6 +68,15 @@ BrowserBox::BrowserBox(unsigned int mode, bool opaque):
     mBackgroundColor = Theme::getThemeColor(Theme::BACKGROUND);
     mHighlightColor = Theme::getThemeColor(Theme::HIGHLIGHT);
     mHyperLinkColor = Theme::getThemeColor(Theme::HYPERLINK);
+    mColors[RED] = Theme::getThemeColor(Theme::RED);
+    mColors[GREEN] = Theme::getThemeColor(Theme::GREEN);
+    mColors[BLUE] = Theme::getThemeColor(Theme::BLUE);
+    mColors[ORANGE] = Theme::getThemeColor(Theme::ORANGE);
+    mColors[YELLOW] = Theme::getThemeColor(Theme::YELLOW);
+    mColors[PINK] = Theme::getThemeColor(Theme::PINK);
+    mColors[PURPLE] = Theme::getThemeColor(Theme::PURPLE);
+    mColors[GRAY] = Theme::getThemeColor(Theme::GRAY);
+    mColors[BROWN] = Theme::getThemeColor(Theme::BROWN);
 }
 
 BrowserBox::~BrowserBox()
@@ -478,15 +487,15 @@ int BrowserBox::calcHeight()
 
                         switch (c)
                         {
-                            case '1': selColor = RED; break;
-                            case '2': selColor = GREEN; break;
-                            case '3': selColor = BLUE; break;
-                            case '4': selColor = ORANGE; break;
-                            case '5': selColor = YELLOW; break;
-                            case '6': selColor = PINK; break;
-                            case '7': selColor = PURPLE; break;
-                            case '8': selColor = GRAY; break;
-                            case '9': selColor = BROWN; break;
+                            case '1': selColor = mColors[RED]; break;
+                            case '2': selColor = mColors[GREEN]; break;
+                            case '3': selColor = mColors[BLUE]; break;
+                            case '4': selColor = mColors[ORANGE]; break;
+                            case '5': selColor = mColors[YELLOW]; break;
+                            case '6': selColor = mColors[PINK]; break;
+                            case '7': selColor = mColors[PURPLE]; break;
+                            case '8': selColor = mColors[GRAY]; break;
+                            case '9': selColor = mColors[BROWN]; break;
                             case '0':
                             default:
                                 selColor = textColor;
diff --git a/src/gui/widgets/browserbox.h b/src/gui/widgets/browserbox.h
index 9969bfd3b..cd69ce56f 100644
--- a/src/gui/widgets/browserbox.h
+++ b/src/gui/widgets/browserbox.h
@@ -147,27 +147,18 @@ class BrowserBox : public gcn::Widget,
             AUTO_WRAP       /**< Maybe it needs a fix or to be redone. */
         };
 
-        /**
-         * BrowserBox colors.
-         *
-         * NOTES (by Javila):
-         *  - color values is "0x" prefix followed by HTML color style.
-         *  - we can add up to 10 different colors: [0..9].
-         *  - not all colors will be fine with all backgrounds due transparent
-         *    windows and widgets. So, I think it's better keep BrowserBox
-         *    opaque (white background) by default.
-         */
         enum
         {
-            RED = 0xff0000,         /**< Color 1 */
-            GREEN = 0x009000,       /**< Color 2 */
-            BLUE = 0x0000ff,        /**< Color 3 */
-            ORANGE = 0xe0980e,      /**< Color 4 */
-            YELLOW = 0xf1dc27,      /**< Color 5 */
-            PINK = 0xff00d8,        /**< Color 6 */
-            PURPLE = 0x8415e2,      /**< Color 7 */
-            GRAY = 0x919191,        /**< Color 8 */
-            BROWN = 0x8e4c17        /**< Color 9 */
+            RED = 0,
+            GREEN = 1,
+            BLUE = 2,
+            ORANGE = 3,
+            YELLOW = 4,
+            PINK = 5,
+            PURPLE = 6,
+            GRAY = 7,
+            BROWN = 8,
+            COLORS_MAX = 9
         };
 
         /**
@@ -229,6 +220,7 @@ class BrowserBox : public gcn::Widget,
         gcn::Color mBackgroundColor;
         gcn::Color mHighlightColor;
         gcn::Color mHyperLinkColor;
+        gcn::Color mColors[COLORS_MAX];
 };
 
 #endif