]> source.dussan.org Git - jquery-ui.git/commitdiff
Datepicker: Rename `select()` method to `_select()`
authorFelix Nagel <info@felixnagel.com>
Sat, 26 Apr 2014 11:30:09 +0000 (13:30 +0200)
committerScott González <scott.gonzalez@gmail.com>
Thu, 29 Jan 2015 22:48:25 +0000 (17:48 -0500)
Make `select()` a private method as it's not part of the specification.

ui/datepicker.js

index 929138c79b2dae10588c4fee33c4730a4894e314..2722c37b6f84af8a662803a32982465eb0034a5f 100644 (file)
@@ -84,7 +84,7 @@ $.widget( "ui.datepicker", {
                        },
                        "click .ui-datepicker-current": function( event ) {
                                event.preventDefault();
-                               this.select( event, new Date().getTime() );
+                               this._select( event, new Date().getTime() );
                        },
                        "click .ui-datepicker-close": function( event ) {
                                event.preventDefault();
@@ -94,7 +94,7 @@ $.widget( "ui.datepicker", {
                                event.preventDefault();
                                // TODO exclude clicks on lead days or handle them correctly
                                // TODO store/read more then just date, also required for multi month picker
-                               this.select( event, $( event.currentTarget ).data( "timestamp" ) );
+                               this._select( event, $( event.currentTarget ).data( "timestamp" ) );
                                if ( this.inline ) {
                                        this.grid.focus();
                                }
@@ -660,7 +660,7 @@ $.widget( "ui.datepicker", {
                }, this.options.position );
        },
 
-       select: function( event, time ) {
+       _select: function( event, time ) {
                this.date.setTime( time ).select();
                this.refresh();
                if ( !this.inline ) {