Ticket #7733: alt_list_color.patch

gui/ListWidget.cpp

@@ -141 +141 @@
         }
 }
 
-void ListWidget::setList(const StringList &list) {
+void ListWidget::setList(const StringList &list, const ColorFlagList *colorFlags) {
         if (_editMode && _caretVisible)
                 drawCaret(true);
 
@@ -150 +150 @@
         _list = list;
         _filter.clear();
         _listIndex.clear();
+        _colorFlags.clear();
+
+        if (colorFlags) {
+                _colorFlags = *colorFlags;
+                assert(_colorFlags.size() == _dataList.size());
+        }
 
         int size = list.size();
         if (_currentPos >= size)
@@ -162 +168 @@
         scrollBarRecalc();
 }
 
-void ListWidget::append(const String &s) {
+void ListWidget::append(const String &s, bool altColor) {
+        if (_dataList.size() == _colorFlags.size())
+                _colorFlags.push_back(altColor);
+
         _dataList.push_back(s);
         _list.push_back(s);
 
@@ -431 +440 @@
 
                 int width;
 
+                bool altColor = false;
+
+                if (!_colorFlags.empty()) {
+                        if (_filter.empty() || _selectedItem == -1)
+                                altColor = _colorFlags[pos];
+                        else
+                                altColor = _colorFlags[_listIndex[pos]];
+                }
+
                 if (_selectedItem == pos && _editMode) {
                         buffer = _editString;
                         adjustOffset();
                         width = _w - r.left - _hlRightPadding - _leftPadding - scrollbarW;
                         g_gui.theme()->drawText(Common::Rect(_x + r.left, y, _x + r.left + width, y + fontHeight - 2),
-                                                                        buffer, _state, Graphics::kTextAlignLeft, inverted, pad, true);
+                                                                        buffer, _state, Graphics::kTextAlignLeft, inverted, pad, true, ThemeEngine::kFontStyleBold,
+                                                                        altColor ? ThemeEngine::kFontColorAlternate : ThemeEngine::kFontColorNormal);
                 } else {
                         buffer = _list[pos];
                         width = _w - r.left - scrollbarW;
                         g_gui.theme()->drawText(Common::Rect(_x + r.left, y, _x + r.left + width, y + fontHeight - 2),
-                                                                        buffer, _state, Graphics::kTextAlignLeft, inverted, pad, true);
+                                                                        buffer, _state, Graphics::kTextAlignLeft, inverted, pad, true, ThemeEngine::kFontStyleBold,
+                                                                        altColor ? ThemeEngine::kFontColorAlternate : ThemeEngine::kFontColorNormal);
                 }
 
                 _textWidth[i] = width;

gui/ListWidget.h

@@ -51 +51 @@
 public:
         typedef Common::String String;
         typedef Common::StringList StringList;
+        typedef Common::Array<bool> ColorFlagList;
 protected:
         StringList              _list;
         StringList              _dataList;
+        ColorFlagList   _colorFlags;
         Common::Array<int>              _listIndex;
         bool                    _editable;
         bool                    _editMode;
@@ -87 +89 @@
 
         virtual Widget *findWidget(int x, int y);
 
-        void setList(const StringList &list);
-        void append(const String &s);
+        void setList(const StringList &list, const ColorFlagList *colorFlags = 0);
+        void append(const String &s, bool altColor = false);
         const StringList &getList()     const                   { return _dataList; }
         int getSelected() const                                         { return (_filter.empty() || _selectedItem == -1) ? _selectedItem : _listIndex[_selectedItem]; }
         void setSelected(int item);

gui/saveload.cpp

@@ -327 +327 @@
         int curSlot = 0;
         int saveSlot = 0;
         StringList saveNames;
+        ListWidget::ColorFlagList colorFlags;
         for (SaveStateList::const_iterator x = _saveList.begin(); x != _saveList.end(); ++x) {
                 // Handle gaps in the list of save games
                 saveSlot = atoi(x->save_slot().c_str());
@@ -335 +336 @@
                                 SaveStateDescriptor dummySave(curSlot, "");
                                 _saveList.insert_at(curSlot, dummySave);
                                 saveNames.push_back(dummySave.description());
+                                colorFlags.push_back(false);
                                 curSlot++;
                         }
 
@@ -349 +351 @@
                 Common::String description = x->description();
                 Common::String trimmedDescription = description;
                 trimmedDescription.trim();
-                if (trimmedDescription.empty())
+                if (trimmedDescription.empty()) {
                         description = "Untitled savestate";
+                        colorFlags.push_back(true);
+                } else {
+                        colorFlags.push_back(false);
+                }
 
                 saveNames.push_back(description);
                 curSlot++;
@@ -362 +368 @@
                 saveNames.push_back(emptyDesc);
                 SaveStateDescriptor dummySave(i, "");
                 _saveList.push_back(dummySave);
+                colorFlags.push_back(false);
         }
 
-        _list->setList(saveNames);
+        _list->setList(saveNames, &colorFlags);
 }
 
 } // End of namespace GUI

gui/themes/default.inc

@@ -1550 +1550 @@
 "color='lightgrey' "
 "/> "
 "<text_color id='color_alternative' "
-"color='green' "
+"color='lightgrey' "
 "/> "
 "<text_color id='color_alternative_inverted' "
-"color='black' "
+"color='255,255,255' "
 "/> "
 "<text_color id='color_alternative_hover' "
-"color='green2' "
+"color='lightgrey' "
 "/> "
 "<text_color id='color_alternative_disabled' "
 "color='lightgrey' "

gui/themes/scummclassic/classic_gfx.stx

@@ -70 +70 @@
                 />
 
                 <text_color     id = 'color_alternative'
-                                color = 'green'
+                                color = 'lightgrey'
                 />
 
                 <text_color     id = 'color_alternative_inverted'
-                                color = 'black'
+                                color = '255, 255, 255'
                 />
 
                 <text_color     id = 'color_alternative_hover'
-                                color = 'green2'
+                                color = 'lightgrey'
                 />
 
                 <text_color     id = 'color_alternative_disabled'

gui/themes/scummmodern/scummmodern_gfx.stx

@@ -130 +130 @@
                 />
 
                 <text_color     id = 'color_alternative'
-                                color = 'black'
+                                color = 'darkgray'
                 />
 
                 <text_color     id = 'color_alternative_inverted'
-                                color = 'black'
+                                color = 'white'
                 />
 
                 <text_color     id = 'color_alternative_hover'
-                                color = 'bgreen'
+                                color = 'darkgray'
                 />
 
                 <text_color     id = 'color_alternative_disabled'
-                                color = '192, 192, 192'
+                                color = 'darkgray'
                 />
 
                 <text_color     id = 'color_button'