Popup: Rename _close to close and use it when selecting something within a popup.

Author: jzaefferer

demos/tooltip/video-player.html

@@ -24,8 +24,12 @@
 		});
 		$(".set").buttonset();
 
-		// TODO hide the tooltip when clicking the button
-		$("ul").menu().popup({
+		$("ul").menu({
+			select: function() {
+				// would also execute some other action here
+				$(this).popup("close");
+			}
+		}).popup({
 			trigger: $(".menu")
 		});
 

tests/visual/menu/popup.html

@@ -25,7 +25,7 @@
 		var selected = {
 			select: function( event, ui ) {
 				$( "<div/>" ).text( "Selected: " + ui.item.text() ).appendTo( "#log" );
-				$(this).hide();
+				$(this).popup("close");
 			}
 		};
 

tests/visual/menu/popup.js

@@ -34,7 +34,7 @@ $.widget( "ui.popup", {
 
 		this.element
 			.addClass("ui-popup")
-		this._close();
+		this.close();
 
 		this._bind(this.options.trigger, {
 			click: function( event ) {
@@ -53,15 +53,15 @@ $.widget( "ui.popup", {
 		this._bind(this.element, {
 			// TODO also triggered when open and clicking the trigger again
 			// figure out how to close in that case, while still closing on regular blur
-			//blur: "_close"
+			//blur: "close"
 		});
 
 		this._bind({
 			// TODO only triggerd on element if it can receive focus
 			// bind to document instead?
 			keyup: function( event ) {
 				if (event.keyCode == $.ui.keyCode.ESCAPE && this.element.is( ":visible" )) {
-					this._close( event );
+					this.close( event );
 					this.options.trigger.focus();
 				}
 			}
@@ -70,7 +70,7 @@ $.widget( "ui.popup", {
 		this._bind(document, {
 			click: function( event ) {
 				if (this.open && !$(event.target).closest(".ui-popup").length) {
-					this._close( event );
+					this.close( event );
 				}
 			}
 		})
@@ -114,7 +114,7 @@ $.widget( "ui.popup", {
 		this._trigger( "open", event );
 	},
 
-	_close: function( event ) {
+	close: function( event ) {
 		this.element
 			.hide()
 			.attr( "aria-hidden", true )