-
-
Notifications
You must be signed in to change notification settings - Fork 593
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed typos in comments #3399
Fixed typos in comments #3399
Conversation
You're at it again! Very impressive! |
You are welcome! |
All my classic typos, like "WIndows" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's great, thanks for the hard work here (and good tools).
I noted a couple of places where it makes no sense to be editing the test code, but does no harm, and thus your tools won't stumble on it next time.
@@ -234,7 +234,7 @@ Drupal.ahah.prototype.success = function (response, status) { | |||
*/ | |||
Drupal.ahah.prototype.error = function (response, uri) { | |||
alert(Drupal.ahahError(response, uri)); | |||
// Resore the previous action and target to the form. | |||
// Restore the previous action and target to the form. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is actual Drupal 6 code, no need to be correcting it here. Not that it matters.
// Restore the previous action and target to the form. | |
// Resore the previous action and target to the form. |
@@ -246,7 +246,7 @@ Drupal.ahah.prototype.error = function (response, uri) { | |||
// Undo hide. | |||
$(this.wrapper).show(); | |||
// Re-enable the element. | |||
$(this.element).removeClass('progess-disabled').attr('disabled', false); | |||
$(this.element).removeClass('progress-disabled').attr('disabled', false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, this is actual Drupal 6 code, no need to correct. I wonder if this bug was ever fixed.
@@ -510,7 +510,7 @@ Drupal.ajax.prototype.commands = { | |||
var effect = ajax.getEffect(response); | |||
|
|||
// We don't know what response.data contains: it might be a string of text | |||
// without HTML, so don't rely on jQuery correctly iterpreting | |||
// without HTML, so don't rely on jQuery correctly interpreting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actual Drupal 7 code. Does no harm to fix here, but irrelevant.
No description provided.