From: Felix Nagel <info@felixnagel.com>
Date: Thu, 3 Dec 2015 17:16:40 +0000 (+0100)
Subject: Calendar: Do not rebuild whole calendar when selecting date
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3cf53089178f14c7c648359c6357adc66cb31595;p=jquery-ui.git

Calendar: Do not rebuild whole calendar when selecting date
---

diff --git a/ui/widgets/calendar.js b/ui/widgets/calendar.js
index e692995f4..b493cb72f 100644
--- a/ui/widgets/calendar.js
+++ b/ui/widgets/calendar.js
@@ -99,7 +99,7 @@ return $.widget( "ui.calendar", {
 			},
 			"mousedown .ui-calendar-calendar button": function( event ) {
 				this._setOption( "value", new Date( $( event.currentTarget ).data( "timestamp" ) ) );
-				this.refresh();
+				this._updateDayElement( "ui-state-active" );
 
 				// Allow datepicker to handle focus
 				if ( this._trigger( "select", event ) !== false ) {
@@ -180,25 +180,29 @@ return $.widget( "ui.calendar", {
 
 			// Check if the needed day is already present in our grid due
 			// to eachDay option changes (eg. other-months demo)
-			return !this.grid.find(
-				"#" + $.ui.escapeSelector( this._getDayId( this.date ) )
-				).length;
+			return !this._getDateElement( this._getDayId( this.date ) ).length;
 		}
 
 		return false;
 	},
 
 	_setActiveDescendant: function() {
+		this.activeDescendant = this._updateDayElement( "ui-state-focus" );
+	},
+
+	_updateDayElement: function( state ) {
 		var id = this._getDayId( this.date );
 
 		this.grid
 			.attr( "aria-activedescendant", id )
-			.find( ".ui-state-focus" )
-			.removeClass( "ui-state-focus" );
+			.find( "button." + state )
+			.removeClass( state );
+
+		return this._getDateElement( id ).children( "button" ).addClass( state );
+	},
 
-		this.activeDescendant = this.grid.find(
-			"#" + $.ui.escapeSelector( id ) + " > button"
-		).addClass( "ui-state-focus" );
+	_getDateElement: function( id ) {
+		return this.grid.find( "#" + $.ui.escapeSelector( id ) );
 	},
 
 	_setLocale: function( locale, dateFormat ) {
@@ -518,7 +522,7 @@ return $.widget( "ui.calendar", {
 	refresh: function() {
 		this.labels = this.options.labels;
 
-		// Determine which day gridcell to focus after refresh
+		// Determine which day grid cell to focus after refresh
 		// TODO: Prevent disabled cells from being focused
 		if ( this.options.numberOfMonths === 1 ) {
 			this.element.find( ".ui-calendar-title" ).html( this._buildTitle() );