Fix a .destroy() bug on ColumnMenu's submenu (#151)

Author: cronvel

CHANGELOG

@@ -1,4 +1,10 @@
 
+v1.49.2
+-------
+
+Fix a .destroy() bug on ColumnMenu's submenu (#151)
+
+
 v1.49.1
 -------
 

lib/document/BaseMenu.js

@@ -148,6 +148,7 @@ BaseMenu.prototype.needInput = true ;
 
 BaseMenu.prototype.destroy = function( isSubDestroy , noDraw = false ) {
 	if ( this.destroyed ) { return ; }
+	if ( this.submenu ) { this.submenu.destroy( true ) ; }
 
 	this.off( 'key' , this.onKey ) ;
 	this.off( 'wheel' , this.onWheel ) ;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "terminal-kit",
-  "version": "1.49.1",
+  "version": "1.49.2",
   "description": "256 colors, keys and mouse, input field, progress bars, screen buffer (including 32-bit composition and image loading), text buffer, and many more... Whether you just need colors and styles, build a simple interactive command line tool or a complexe terminal app: this is the absolute terminal lib for Node.js!",
   "main": "lib/termkit.js",
   "directories": {

sample/document/column-menu-submenu-test.js

@@ -132,7 +132,17 @@ var columnMenu = new termkit.ColumnMenu( {
 				} ,
 				{
 					content: 'Manual' ,
-					value: 'manual'
+					value: 'manual' ,
+					items: [
+						{
+							content: 'Local' ,
+							value: 'local'
+						} ,
+						{
+							content: 'Online' ,
+							value: 'online'
+						}
+					]
 				}
 			]
 		}