From: Jerome St-Louis <jerome@ecere.com>
Date: Sat, 13 Sep 2014 03:48:52 +0000 (-0400)
Subject: ecere/gui/controls/Menu: copyText option in Menu; Fixed activation issue on 3-levels... 
X-Git-Tag: 0.44.11~37
X-Git-Url: http://ecere.com/cgi-bin/gitweb.cgi?p=sdk;a=commitdiff_plain;h=62e7c744359795cd728e30ba0acbb0afd785f193

ecere/gui/controls/Menu: copyText option in Menu; Fixed activation issue on 3-levels popups
---

diff --git a/ecere/src/gui/controls/Menu.ec b/ecere/src/gui/controls/Menu.ec
index d3d5185..9714795 100644
--- a/ecere/src/gui/controls/Menu.ec
+++ b/ecere/src/gui/controls/Menu.ec
@@ -604,9 +604,10 @@ public:
    }
 
    property Menu parent { set { if(value) value.AddSubMenu(this); } };
-   property const char * text { set { text = (char *)value; /* CopyString(value);*/ } };
+   property const char * text { set { if(copyText) delete text; text = copyText ? CopyString(value) : (char *)value; } };
    property Key hotKey { set { hotKey = value; } };
    property bool hasMargin { set { hasMargin = value; } };
+   property bool copyText { set { copyText = value; } };
 
 private:
    OldList items;
@@ -618,6 +619,7 @@ private:
    int itemCount;
    bool mergeClients;
    bool hasMargin;
+   bool copyText;
 
    Menu()
    {
@@ -626,6 +628,7 @@ private:
 
    ~Menu()
    {
+      if(copyText) delete text;
       Clean(null);
       Clear();
    }
@@ -682,7 +685,10 @@ public class PopupMenu : Window
          }
          master.Update(null);
       }
+      incref window;
+      window.master.Activate();
       result = window.Destroy(0);
+      delete window;
       // This looks like a hack...
       RestoreCaret();
       return result;