Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix comment typos. #853

Closed
wants to merge 1 commit into from

3 participants

@faulkner

No description provided.

@khiltd khiltd commented on the diff
src/css.js
@@ -374,15 +374,15 @@ function augmentWidthOrHeight( elem, name, extra, isBorderBox ) {
val -= parseFloat( curCSS( elem, "padding" + cssExpand[ i ] ) ) || 0;
}
- // at this point, extra isnt border nor margin, so remove border
+ // at this point, extra isn't border nor margin, so remove border
@khiltd
khiltd added a note

That ungood grammar still is

Yeah, I cringed at that, but if I started on grammar I'd be busy for a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@rwaldron rwaldron closed this in 7b5ffcd
@mescoda mescoda referenced this pull request from a commit in mescoda/jquery
@faulkner faulkner Fixes a variety of typographical problems. Closes gh-853 9c55fff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 11, 2012
  1. @faulkner

    Fix comment typos.

    faulkner authored
This page is out of date. Refresh to see the latest.
View
24 speed/jquery-basis.js
@@ -519,7 +519,7 @@ jQuery.extend({
noop: function() {},
- // Evalulates a script in a global context
+ // Evaluates a script in a global context
globalEval: function( data ) {
if ( data && rnotwhite.test(data) ) {
// Inspired by code by Andrea Giammarchi
@@ -1308,7 +1308,7 @@ jQuery.fn.extend({
classNames = value.split( rspace );
while ( (className = classNames[ i++ ]) ) {
- // check each className given, space seperated list
+ // check each className given, space separated list
state = isBool ? state : !self.hasClass( className );
self[ state ? "addClass" : "removeClass" ]( className );
}
@@ -1362,7 +1362,7 @@ jQuery.fn.extend({
var option = options[ i ];
if ( option.selected ) {
- // Get the specifc value for the option
+ // Get the specific value for the option
value = jQuery(option).val();
// We don't need an array for one selects
@@ -2634,7 +2634,7 @@ var chunker = /((?:\((?:\([^()]+\)|[^()]+)+\)|\[(?:\[[^[\]]*\]|['"][^'"]*['"]|[^
baseHasDuplicate = true;
// Here we check if the JavaScript engine is using some sort of
-// optimization where it does not always call our comparision
+// optimization where it does not always call our comparison
// function. If that is the case, discard the hasDuplicate value.
// Thus far that includes Google Chrome.
[0, 0].sort(function(){
@@ -3413,7 +3413,7 @@ if ( document.documentElement.compareDocumentPosition ) {
};
}
-// Utility function for retreiving the text value of an array of DOM nodes
+// Utility function for retrieving the text value of an array of DOM nodes
function getText( elems ) {
var ret = "", elem;
@@ -4147,12 +4147,12 @@ jQuery.fn.extend({
if ( !jQuery.support.noCloneEvent && !jQuery.isXMLDoc(this) ) {
// IE copies events bound via attachEvent when
// using cloneNode. Calling detachEvent on the
- // clone will also remove the events from the orignal
+ // clone will also remove the events from the original.
// In order to get around this, we use innerHTML.
// Unfortunately, this means some modifications to
// attributes in IE that are actually only stored
// as properties will not be copied (such as the
- // the name attribute on an input).
+ // name attribute on an input).
var html = this.outerHTML, ownerDocument = this.ownerDocument;
if ( !html ) {
var div = ownerDocument.createElement("div");
@@ -4884,7 +4884,7 @@ jQuery.each( "ajaxStart ajaxStop ajaxComplete ajaxError ajaxSuccess ajaxSend".sp
jQuery.extend({
get: function( url, data, callback, type ) {
- // shift arguments if data argument was omited
+ // shift arguments if data argument was omitted
if ( jQuery.isFunction( data ) ) {
type = type || callback;
callback = data;
@@ -4909,7 +4909,7 @@ jQuery.extend({
},
post: function( url, data, callback, type ) {
- // shift arguments if data argument was omited
+ // shift arguments if data argument was omitted
if ( jQuery.isFunction( data ) ) {
type = type || callback;
callback = data;
@@ -4946,7 +4946,7 @@ jQuery.extend({
// Create the request object; Microsoft failed to properly
// implement the XMLHttpRequest in IE7 (can't request local files),
// so we use the ActiveXObject when it is available
- // This function can be overriden by calling jQuery.ajaxSetup
+ // This function can be overridden by calling jQuery.ajaxSetup
xhr: window.XMLHttpRequest && (window.location.protocol !== "file:" || !window.ActiveXObject) ?
function() {
return new window.XMLHttpRequest();
@@ -5148,7 +5148,7 @@ jQuery.extend({
jQuery.event.trigger( "ajaxStop" );
}
- // close opended socket
+ // close opened socket
xhr.abort();
return false;
}
@@ -5254,7 +5254,7 @@ jQuery.extend({
complete();
}
- // firefox 1.5 doesn't fire statechange for sync requests
+ // Firefox 1.5 doesn't fire statechange for sync requests
if ( !s.async ) {
onreadystatechange();
}
View
2  src/ajax.js
@@ -709,7 +709,7 @@ jQuery.extend({
}
- // aborting is no longer a cancelation
+ // aborting is no longer a cancellation
strAbort = "abort";
// Install callbacks on deferreds
View
2  src/ajax/xhr.js
@@ -111,7 +111,7 @@ if ( jQuery.support.ajax ) {
xml;
// Firefox throws exceptions when accessing properties
- // of an xhr when a network error occured
+ // of an xhr when a network error occurred
// http://helpful.knobs-dials.com/index.php/Component_returned_failure_code:_0x80040111_(NS_ERROR_NOT_AVAILABLE)
try {
View
2  src/attributes.js
@@ -122,7 +122,7 @@ jQuery.fn.extend({
classNames = value.split( core_rspace );
while ( (className = classNames[ i++ ]) ) {
- // check each className given, space seperated list
+ // check each className given, space separated list
state = isBool ? state : !self.hasClass( className );
self[ state ? "addClass" : "removeClass" ]( className );
}
View
6 src/css.js
@@ -374,15 +374,15 @@ function augmentWidthOrHeight( elem, name, extra, isBorderBox ) {
val -= parseFloat( curCSS( elem, "padding" + cssExpand[ i ] ) ) || 0;
}
- // at this point, extra isnt border nor margin, so remove border
+ // at this point, extra isn't border nor margin, so remove border
@khiltd
khiltd added a note

That ungood grammar still is

Yeah, I cringed at that, but if I started on grammar I'd be busy for a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
if ( extra !== "margin" ) {
val -= parseFloat( curCSS( elem, "border" + cssExpand[ i ] + "Width" ) ) || 0;
}
} else {
- // at this point, extra isnt content, so add padding
+ // at this point, extra isn't content, so add padding
val += parseFloat( curCSS( elem, "padding" + cssExpand[ i ] ) ) || 0;
- // at this point, extra isnt content nor padding, so add border
+ // at this point, extra isn't content nor padding, so add border
if ( extra !== "padding" ) {
val += parseFloat( curCSS( elem, "border" + cssExpand[ i ] + "Width" ) ) || 0;
}
View
2  src/effects.js
@@ -404,7 +404,7 @@ Tween.propHooks = {
return tween.elem[ tween.prop ];
}
- // passing any value as a 4th paramter to .css will automatically
+ // passing any value as a 4th parameter to .css will automatically
// attempt a parseFloat and fallback to a string if the parse fails
// so, simple values such as "10px" are parsed to Float.
// complex values such as "rotate(1rad)" are returned as is.
Something went wrong with that request. Please try again.