#179 checkbox boolean support added

dave · dave · commit 48acd73c2b61 · 2023-01-09T21:35:50.000Z
diff --git a/examples/avr/keyboardEthernetShield/keyboardEthernetShield.emf b/examples/avr/keyboardEthernetShield/keyboardEthernetShield.emf
@@ -382,7 +382,7 @@
       "type": "boolItem",
       "defaultValue": "false",
       "item": {
-        "naming": "YES_NO",
+        "naming": "CHECKBOX",
         "name": "Flag3",
         "variableName": "AdditionalBoolFlagFlag3",
         "id": 27,
@@ -397,7 +397,7 @@
       "type": "boolItem",
       "defaultValue": "false",
       "item": {
-        "naming": "YES_NO",
+        "naming": "CHECKBOX",
         "name": "Flag4",
         "variableName": "AdditionalBoolFlagFlag4",
         "id": 28,
@@ -747,8 +747,8 @@
     "eepromDefinition": "avr:",
     "authenticatorDefinition": "rom:100:6",
     "projectIoExpanders": [
-      "deviceIO:",
-      "mcp23017:io23017:32:2"
+      "mcp23017:io23017:32:2",
+      "deviceIO:"
     ],
     "menuInMenuCollection": {
       "menuDefinitions": []
diff --git a/examples/avr/keyboardEthernetShield/keyboardEthernetShield.ino b/examples/avr/keyboardEthernetShield/keyboardEthernetShield.ino
@@ -88,6 +88,11 @@ void setup() {
     // Here we set the character to be used for back, next and editing for the "cursor".
     renderer.setEditorChars(0b01111111, 0b01111110, '=');
 
+    // You can also have title widgets on LCDs, maximum of 7 states. They use up the custom chars 0..7. How it works
+    // is that you define a custom character for each icon, and treat them just as you would a regular icon. The height
+    // of the widget is fixed, and the height field instead indicates the first custom character for the first icon.
+    renderer.setFirstWidget(&connectedWidget);
+
 	setupMenu();
 
 	auto* authenticator = reinterpret_cast<EepromAuthenticatorManager*>(menuMgr.getAuthenticator());
@@ -127,7 +132,6 @@ void setup() {
     // and lastly we register a communication listener, it updates the title widget
     // that shows connectivity state on the right corner.
     menuConnectivityIoTMonitor.registerCommsNotification(onCommsChange);
-    renderer.setFirstWidget(&connectedWidget);
 
     // and finally, when the display times out, take over and draw a custom screen
     renderer.setResetCallback([] {
diff --git a/examples/avr/keyboardEthernetShield/keyboardEthernetShield_menu.cpp b/examples/avr/keyboardEthernetShield/keyboardEthernetShield_menu.cpp
@@ -44,9 +44,9 @@ RENDERING_CALLBACK_NAME_INVOKE(fnRomChoicesRtCall, backSubItemRenderFn, "Rom Cho
 const PROGMEM SubMenuInfo minfoRomChoices = { "Rom Choices", 20, 0xffff, 0, NO_CALLBACK };
 BackMenuItem menuBackRomChoices(fnRomChoicesRtCall, &menuRomChoicesItemNum);
 SubMenuItem menuRomChoices(&minfoRomChoices, &menuBackRomChoices, NULL);
-const PROGMEM BooleanMenuInfo minfoAdditionalBoolFlagFlag4 = { "Flag4", 28, 41, 1, NO_CALLBACK, NAMING_YES_NO };
+const PROGMEM BooleanMenuInfo minfoAdditionalBoolFlagFlag4 = { "Flag4", 28, 41, 1, NO_CALLBACK, NAMING_CHECKBOX };
 BooleanMenuItem menuAdditionalBoolFlagFlag4(&minfoAdditionalBoolFlagFlag4, false, NULL);
-const PROGMEM BooleanMenuInfo minfoAdditionalBoolFlagFlag3 = { "Flag3", 27, 40, 1, NO_CALLBACK, NAMING_YES_NO };
+const PROGMEM BooleanMenuInfo minfoAdditionalBoolFlagFlag3 = { "Flag3", 27, 40, 1, NO_CALLBACK, NAMING_CHECKBOX };
 BooleanMenuItem menuAdditionalBoolFlagFlag3(&minfoAdditionalBoolFlagFlag3, false, &menuAdditionalBoolFlagFlag4);
 const PROGMEM BooleanMenuInfo minfoAdditionalBoolFlagFlag2 = { "Flag2", 26, 39, 1, NO_CALLBACK, NAMING_ON_OFF };
 BooleanMenuItem menuAdditionalBoolFlagFlag2(&minfoAdditionalBoolFlagFlag2, false, &menuAdditionalBoolFlagFlag3);
@@ -113,8 +113,8 @@ void setupMenu() {
     authManager.initialise(menuMgr.getEepromAbstraction(), 100);
     menuMgr.setAuthenticator(&authManager);
     // Now add any readonly, non-remote and visible flags.
-    menuConnectivityIoTMonitor.setLocalOnly(true);
     menuConnectivityAuthenticator.setLocalOnly(true);
+    menuConnectivityIoTMonitor.setLocalOnly(true);
     menuConnectivity.setLocalOnly(true);
     menuConnectivity.setSecured(true);
     menuHiddenItem.setVisible(false);
diff --git a/examples/avr/keyboardEthernetShield/keyboardEthernetShield_menu.h b/examples/avr/keyboardEthernetShield/keyboardEthernetShield_menu.h
@@ -21,8 +21,8 @@
 #include <ScrollChoiceMenuItem.h>
 #include <RemoteMenuItem.h>
 #include <EditableLargeNumberMenuItem.h>
-#include <IoAbstraction.h>
 #include <IoAbstractionWire.h>
+#include <IoAbstraction.h>
 #include <EepromAbstraction.h>
 #include <RemoteAuthentication.h>
 #include "tcMenuLiquidCrystal.h"
diff --git a/examples/esp/esp32s2Saola/esp32s2Saola.emf b/examples/esp/esp32s2Saola/esp32s2Saola.emf
@@ -153,6 +153,21 @@
         "visible": true
       }
     },
+    {
+      "parentId": 6,
+      "type": "boolItem",
+      "defaultValue": "true",
+      "item": {
+        "naming": "CHECKBOX",
+        "name": "New BoolItem",
+        "variableName": "SelectMeNewBoolItem",
+        "id": 21,
+        "eepromAddress": -1,
+        "readOnly": false,
+        "localOnly": false,
+        "visible": true
+      }
+    },
     {
       "parentId": 0,
       "type": "subMenu",
diff --git a/examples/esp/esp32s2Saola/esp32s2Saola.ino b/examples/esp/esp32s2Saola/esp32s2Saola.ino
@@ -13,7 +13,7 @@
 #include <tcMenuVersion.h>
 #include <WiFi.h>
 #include <stockIcons/wifiAndConnectionIcons16x12.h>
-
+#include <tcUtil.h>
 
 #define MENU_WIFIMODE_STATION 0
 const char pgmsListHeader[] PROGMEM = "List items";
@@ -45,19 +45,9 @@ void setup() {
     startWiFiAndListener();
     renderer.setFirstWidget(&wifiWidget);
 
-    // lastly when the title is pressed present a dialog.
+    // lastly we capture when the root title is pressed present a standard version dialog.
     setTitlePressedCallback([](int titleCb) {
-        auto dlg = renderer.getDialog();
-        if(!dlg->isInUse()) {
-            dlg->setButtons(BTNTYPE_NONE, BTNTYPE_CLOSE);
-            dlg->show(applicationInfo.name, false);
-            char sz[25];
-            char ver[10];
-            tccore::copyTcMenuVersion(ver, sizeof ver);
-            strcpy(sz, "TcMenu lib ");
-            strcat(sz, ver);
-            dlg->copyIntoBuffer(sz);
-        }
+        showVersionDialog(&applicationInfo);
     });
 }
 
diff --git a/examples/esp/esp32s2Saola/esp32s2Saola_menu.cpp b/examples/esp/esp32s2Saola/esp32s2Saola_menu.cpp
@@ -54,8 +54,10 @@ RENDERING_CALLBACK_NAME_INVOKE(fnExtrasRtCall, backSubItemRenderFn, "Extras", -1
 const PROGMEM SubMenuInfo minfoExtras = { "Extras", 10, 0xffff, 0, NO_CALLBACK };
 BackMenuItem menuBackExtras(fnExtrasRtCall, &menuExtrasText);
 SubMenuItem menuExtras(&minfoExtras, &menuBackExtras, &menuConnectivity);
+const PROGMEM BooleanMenuInfo minfoSelectMeNewBoolItem = { "New BoolItem", 21, 0xffff, 1, NO_CALLBACK, NAMING_CHECKBOX };
+BooleanMenuItem menuSelectMeNewBoolItem(&minfoSelectMeNewBoolItem, true, NULL);
 const PROGMEM AnyMenuInfo minfoSelectMePressMe = { "Press Me", 9, 0xffff, 0, pressMeActionRun };
-ActionMenuItem menuSelectMePressMe(&minfoSelectMePressMe, NULL);
+ActionMenuItem menuSelectMePressMe(&minfoSelectMePressMe, &menuSelectMeNewBoolItem);
 const PROGMEM FloatMenuInfo minfoSelectMeFloat2 = { "Float 2", 8, 0xffff, 3, NO_CALLBACK };
 FloatMenuItem menuSelectMeFloat2(&minfoSelectMeFloat2, 0.0, &menuSelectMePressMe);
 const PROGMEM FloatMenuInfo minfoSelectMeFloat1 = { "Float 1", 7, 0xffff, 3, NO_CALLBACK };
diff --git a/examples/esp/esp32s2Saola/esp32s2Saola_menu.h b/examples/esp/esp32s2Saola/esp32s2Saola_menu.h
@@ -48,6 +48,7 @@ extern Rgb32MenuItem menuExtrasColor;
 extern TextMenuItem menuExtrasText;
 extern BackMenuItem menuBackExtras;
 extern SubMenuItem menuExtras;
+extern BooleanMenuItem menuSelectMeNewBoolItem;
 extern ActionMenuItem menuSelectMePressMe;
 extern FloatMenuItem menuSelectMeFloat2;
 extern FloatMenuItem menuSelectMeFloat1;
diff --git a/examples/mbed/stm32f429FrameBuffer/stm32f429FrameBuffer.emf b/examples/mbed/stm32f429FrameBuffer/stm32f429FrameBuffer.emf
@@ -251,6 +251,21 @@
         "visible": true
       }
     },
+    {
+      "parentId": 11,
+      "type": "boolItem",
+      "defaultValue": "false",
+      "item": {
+        "naming": "CHECKBOX",
+        "name": "Bool Check",
+        "variableName": "SamplesBoolCheck",
+        "id": 22,
+        "eepromAddress": -1,
+        "readOnly": false,
+        "localOnly": false,
+        "visible": true
+      }
+    },
     {
       "parentId": 0,
       "type": "actionMenu",
@@ -357,7 +372,7 @@
       },
       {
         "name": "ITEM_FONT",
-        "latestValue": "ada:OpenSans18,0",
+        "latestValue": "ada:OpenSansCyrillicLatin18,0",
         "subsystem": "THEME"
       },
       {
diff --git a/examples/mbed/stm32f429FrameBuffer/stm32f429FrameBuffer_menu.cpp b/examples/mbed/stm32f429FrameBuffer/stm32f429FrameBuffer_menu.cpp
@@ -37,8 +37,10 @@ BackMenuItem menuBackUnicode(fnUnicodeRtCall, &menuUnicodeChoice);
 SubMenuItem menuUnicode(&minfoUnicode, &menuBackUnicode, NULL);
 const AnyMenuInfo minfoDialogs = { "Dialogs", 18, 0xffff, 0, onPresentDialog };
 ActionMenuItem menuDialogs(&minfoDialogs, &menuUnicode);
+const BooleanMenuInfo minfoSamplesBoolCheck = { "Bool Check", 22, 0xffff, 1, NO_CALLBACK, NAMING_CHECKBOX };
+BooleanMenuItem menuSamplesBoolCheck(&minfoSamplesBoolCheck, false, NULL);
 RENDERING_CALLBACK_NAME_INVOKE(fnSamplesLgePosRtCall, largeNumItemRenderFn, "LgePos", -1, NO_CALLBACK)
-EditableLargeNumberMenuItem menuSamplesLgePos(fnSamplesLgePosRtCall, LargeFixedNumber(7, 0, 12456U, 0U, false), 17, false, NULL);
+EditableLargeNumberMenuItem menuSamplesLgePos(fnSamplesLgePosRtCall, LargeFixedNumber(7, 0, 12456U, 0U, false), 17, false, &menuSamplesBoolCheck);
 RENDERING_CALLBACK_NAME_INVOKE(fnSamplesRGBRtCall, rgbAlphaItemRenderFn, "RGB", -1, NO_CALLBACK)
 Rgb32MenuItem menuSamplesRGB(fnSamplesRGBRtCall, RgbColor32(255, 170, 187), 16, false, &menuSamplesLgePos);
 RENDERING_CALLBACK_NAME_INVOKE(fnSamplesTimeRtCall, timeItemRenderFn, "Time", -1, NO_CALLBACK)
@@ -93,6 +95,6 @@ void setupMenu() {
     renderer.setTitleMode(BaseGraphicalRenderer::TITLE_FIRST_ROW);
     renderer.setUseSliderForAnalog(true);
     renderer.enableTcUnicode();
-    installCoolBlueModernTheme(renderer, MenuFontDef(OpenSansCyrillicLatin18, 0), MenuFontDef(&RobotoMedium24, 1), false);
+    installCoolBlueModernTheme(renderer, MenuFontDef(&OpenSansCyrillicLatin18, 0), MenuFontDef(&RobotoMedium24, 1), false);
 }
 
diff --git a/examples/mbed/stm32f429FrameBuffer/stm32f429FrameBuffer_menu.h b/examples/mbed/stm32f429FrameBuffer/stm32f429FrameBuffer_menu.h
@@ -42,6 +42,7 @@ extern EnumMenuItem menuUnicodeChoice;
 extern BackMenuItem menuBackUnicode;
 extern SubMenuItem menuUnicode;
 extern ActionMenuItem menuDialogs;
+extern BooleanMenuItem menuSamplesBoolCheck;
 extern EditableLargeNumberMenuItem menuSamplesLgePos;
 extern Rgb32MenuItem menuSamplesRGB;
 extern TimeFormattedMenuItem menuSamplesTime;
diff --git a/examples/mbed/stm32f429FrameBuffer/tcMenuStChromaArt.cpp b/examples/mbed/stm32f429FrameBuffer/tcMenuStChromaArt.cpp
@@ -105,35 +105,6 @@ void StChromaArtDrawable::drawPolygon(const Coord *points, int numPoints, bool f
 void StChromaArtDrawable::transaction(bool isStarting, bool redrawNeeded) {
 }
 
-/**
- * The lightest weight plotter possible for the BSP driver, implements the needed functions direct to BSP calls.
- */
-class BspTextPlotPipeline : public TextPlotPipeline {
-private:
-    Coord cursorPos {};
-public:
-    void drawPixel(uint16_t x, uint16_t y, uint32_t color) override {
-        BSP_LCD_DrawPixel(x, y, color);
-    }
-
-    void setCursor(const Coord &where) override {
-        cursorPos = where;
-    }
-
-    Coord getCursor() override {
-        return cursorPos;
-    }
-
-    Coord getDimensions() override {
-        return Coord(BSP_LCD_GetXSize(), BSP_LCD_GetYSize());
-    }
-};
-
-UnicodeFontHandler *StChromaArtDrawable::createFontHandler() {
-    auto fontPlotter = new BspTextPlotPipeline();
-    return new UnicodeFontHandler(fontPlotter, ENCMODE_UTF8);
-}
-
 #if TC_BSP_TOUCH_DEVICE_PRESENT == true
 
 iotouch::TouchState StBspTouchInterrogator::internalProcessTouch(float *ptrX, float *ptrY, iotouch::TouchInterrogator::TouchRotation rotation,
diff --git a/examples/mbed/stm32f429FrameBuffer/tcMenuStChromaArt.h b/examples/mbed/stm32f429FrameBuffer/tcMenuStChromaArt.h
@@ -61,9 +61,6 @@ class StChromaArtDrawable : public DeviceDrawable {
     void transaction(bool isStarting, bool redrawNeeded) override;
     Coord internalTextExtents(const void *font, int mag, const char *text, int *baseline) override;
     void drawPixel(uint16_t x, uint16_t y) override { BSP_LCD_DrawPixel(x, y, drawColor); }
-
-protected:
-    UnicodeFontHandler *createFontHandler() override;
 };
 
 #if TC_BSP_TOUCH_DEVICE_PRESENT == true
diff --git a/src/MenuItems.cpp b/src/MenuItems.cpp
@@ -314,6 +314,9 @@ void copyMenuItemNameAndValue(const MenuItem* item, char* buffer, size_t bufferS
     copyMenuItemValue(item, buffer + pos, bufferSize - pos);
 }
 
+const char CHECKED_STR[] PROGMEM = "[X]";
+const char UNCHECKED_STR[] PROGMEM = "[ ]";
+
 void copyMenuItemValue(const MenuItem* item, char* buffer, size_t bufferSize) {
     buffer[0] = 0;
     if(item->getMenuType() == MENUTYPE_ENUM_VALUE) {
@@ -331,6 +334,9 @@ void copyMenuItemValue(const MenuItem* item, char* buffer, size_t bufferSize) {
             case NAMING_YES_NO:
                 val = boolItem->getBoolean() ? YES_STR : NO_STR;
                 break;
+            case NAMING_CHECKBOX:
+                val = boolItem->getBoolean() ? CHECKED_STR : UNCHECKED_STR;
+                break;
             default:
                 val = boolItem->getBoolean() ? TRUE_STR : FALSE_STR;
                 break;
diff --git a/src/MenuItems.h b/src/MenuItems.h
@@ -136,7 +136,9 @@ enum BooleanNaming : uint8_t {
 	/** prefer the use of On / Off */
 	NAMING_ON_OFF,
 	/** prefer the user of Yes and No */
-	NAMING_YES_NO
+	NAMING_YES_NO,
+    /** prefer that the user see a checkbox */
+    NAMING_CHECKBOX
 };
 
 /**
diff --git a/src/graphics/BaseGraphicalRenderer.cpp b/src/graphics/BaseGraphicalRenderer.cpp
@@ -430,8 +430,7 @@ ItemDisplayProperties::ComponentType BaseGraphicalRenderer::toComponentType(Grid
         case GridPosition::DRAW_AS_ICON_ONLY:
         case GridPosition::DRAW_AS_ICON_TEXT:
         default:
-            return isItemActionable(pMenuItem) || pMenuItem->getMenuType() == MENUTYPE_BOOLEAN_VALUE
-                   ? ItemDisplayProperties::COMPTYPE_ACTION : ItemDisplayProperties::COMPTYPE_ITEM;
+            return isItemActionable(pMenuItem) ? ItemDisplayProperties::COMPTYPE_ACTION : ItemDisplayProperties::COMPTYPE_ITEM;
     }
 }
 
diff --git a/src/graphics/GraphicsDeviceRenderer.cpp b/src/graphics/GraphicsDeviceRenderer.cpp
@@ -145,6 +145,13 @@ namespace tcgfx {
         bool weAreEditingWithCursor = pEntry->getMenuItem()->isEditing() && menuMgr.getCurrentEditor() != nullptr
                                       && editorHintNeedsCursor(menuMgr.getEditorHints().getEditorRenderingType());
 
+        bool valueNeeded = true;
+        if(pEntry->getMenuItem()->getMenuType() == MENUTYPE_BOOLEAN_VALUE) {
+            if (reinterpret_cast<BooleanMenuItem *>(pEntry->getMenuItem())->getBooleanNaming() == NAMING_CHECKBOX) {
+                valueNeeded = false;
+            }
+        }
+
         if(just == GridPosition::JUSTIFY_TITLE_LEFT_VALUE_RIGHT || weAreEditingWithCursor) {
             // special case, title left, value right.
             Coord wh = Coord(where.x + padding.left, where.y + padding.top);
@@ -153,7 +160,9 @@ namespace tcgfx {
             drawable->setDrawColor(fg);
             drawable->drawText(wh, props->getFont(), props->getFontMagnification(), buffer);
 
-            copyMenuItemValue(pEntry->getMenuItem(), buffer, bufferSize);
+            if(valueNeeded) {
+                copyMenuItemValue(pEntry->getMenuItem(), buffer, bufferSize);
+            } else buffer[0] = 0;
             int16_t right = where.x + size.x - (drawable->textExtents(props->getFont(), props->getFontMagnification(), buffer).x + padding.right);
             wh.x = right;
             if(weAreEditingWithCursor) {
@@ -174,7 +183,6 @@ namespace tcgfx {
         else {
             char sz[32];
             bool nameNeeded = itemNeedsName(just);
-            bool valueNeeded = itemNeedsValue(just);
             if(valueNeeded && nameNeeded) {
                 copyMenuItemNameAndValue(pEntry->getMenuItem(), sz, sizeof sz, 0);
             } else if(valueNeeded) {
@@ -285,6 +293,29 @@ namespace tcgfx {
         }
     }
 
+    void GraphicsDeviceRenderer::drawCheckbox(GridPositionRowCacheEntry *entry, Coord& where, Coord& size) {
+        auto padding = entry->getDisplayProperties()->getPadding();
+        auto* icon = propertiesFactory.iconForMenuItem(entry->getMenuItem()->isEditing() ? SPECIAL_ID_EDIT_ICON :SPECIAL_ID_ACTIVE_ICON);
+
+        drawCoreLineItem(entry, icon, where, size, true);
+        auto hei = size.y - (padding.top + padding.top);
+        auto startingX = where.x + size.x - (padding.left + padding.right + hei);
+        auto boolItem = reinterpret_cast<BooleanMenuItem*>(entry->getMenuItem());
+        auto hl = entry->getDisplayProperties()->getColor(ItemDisplayProperties::HIGHLIGHT1);
+        auto txtCol = entry->getDisplayProperties()->getColor(ItemDisplayProperties::TEXT);
+
+        drawable->setDrawColor(txtCol);
+        drawable->drawBox(Coord(startingX, where.y + padding.top), Coord(hei, hei), false);
+        if(hl != txtCol) {
+            drawable->drawBox(Coord(startingX + 1, where.y + padding.top + 1), Coord(hei - 2, hei - 2), false);
+        }
+        if(boolItem->getBoolean()) {
+            drawable->setDrawColor(hl);
+            drawable->drawBox(Coord(startingX + 2, where.y + padding.top + 2), Coord(hei - 4, hei - 4), true);
+        }
+        internalDrawText(entry, where, Coord(size.x - (hei + padding.left), size.y));
+    }
+
     void GraphicsDeviceRenderer::drawUpDownItem(GridPositionRowCacheEntry *entry, Coord& where, Coord& size) {
         auto padding = entry->getDisplayProperties()->getPadding();
         auto* icon = propertiesFactory.iconForMenuItem(entry->getMenuItem()->isEditing() ? SPECIAL_ID_EDIT_ICON :SPECIAL_ID_ACTIVE_ICON);
@@ -315,6 +346,13 @@ namespace tcgfx {
     }
 
     void GraphicsDeviceRenderer::drawTextualItem(GridPositionRowCacheEntry* pEntry, Coord& where, Coord& size) {
+        if(pEntry->getMenuItem()->getMenuType() == MENUTYPE_BOOLEAN_VALUE) {
+            auto boolItem = reinterpret_cast<BooleanMenuItem*>(pEntry->getMenuItem());
+            if(boolItem->getBooleanNaming() == NAMING_CHECKBOX) {
+                drawCheckbox(pEntry, where, size);
+                return;
+            }
+        }
         auto* icon = propertiesFactory.iconForMenuItem(pEntry->getMenuItem()->isEditing() ? SPECIAL_ID_EDIT_ICON :SPECIAL_ID_ACTIVE_ICON);
         drawCoreLineItem(pEntry, icon, where, size, true);
         internalDrawText(pEntry, Coord(where.x, where.y), Coord(size.x, size.y));
diff --git a/src/graphics/GraphicsDeviceRenderer.h b/src/graphics/GraphicsDeviceRenderer.h
@@ -345,6 +345,7 @@ namespace tcgfx {
         void internalDrawText(GridPositionRowCacheEntry* pEntry, const Coord& where, const Coord& size);
         void drawCoreLineItem(GridPositionRowCacheEntry* entry, DrawableIcon* icon, Coord &where, Coord &size, bool drawBg);
         void drawTextualItem(GridPositionRowCacheEntry* entry, Coord& where, Coord& size);
+        void drawCheckbox(GridPositionRowCacheEntry *entry, Coord& where, Coord& size);
         void drawSlider(GridPositionRowCacheEntry* entry, AnalogMenuItem* pItem, Coord& where, Coord& size);
         void drawUpDownItem(GridPositionRowCacheEntry* entry, Coord& where, Coord& size);
         void drawIconItem(GridPositionRowCacheEntry *pEntry, Coord& where, Coord& size);
