line 223 ahah.js $(this.element).removeClass('progess-disabled').attr('disabled', false);

yes, progess instead progress because at line 122 was added class progress-disabled - $(this.element).addClass('progress-disabled').attr('disabled', true);

in this case we don't remove class progress disabled, trying to remove progess

:)

CommentFileSizeAuthor
#6 ahah_typo-1567530-6.patch416 bytesmdupont
#4 ahah_typo-1567530-4-D6.patch448 bytesmdupont
#1 ahah_typo-1567530-1.patch448 bytesmdupont

Comments

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
Version: 6.26 » 6.x-dev
Priority: Major » Normal
Status: Active » Needs review
Issue tags: -Progress Bar
StatusFileSize
new ahah_typo-1567530-1.patch448 bytes

Confirmed, still here in 6.x-dev, but it's fixed in D7. Straightforward patch attached.

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
Title: .removeClass('progess-disabled') » Typo in ahah.js: wrong class name

Status: Needs review » Needs work

The last submitted patch, ahah_typo-1567530-1.patch, failed testing.

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
Status: Needs work » Needs review
StatusFileSize
new ahah_typo-1567530-4-D6.patch448 bytes

Renamed the patch file to avoid triggering the test bot since this is a D6 patch.

Status: Needs review » Needs work

The last submitted patch, ahah_typo-1567530-4-D6.patch, failed testing.

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
StatusFileSize
new ahah_typo-1567530-6.patch416 bytes

My mistake, wrong path in patch. This one should pass.

mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
Status: Needs work » Needs review
mdupont’s picture

mdupont
Primary language French
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Being bold since this is really a no-brainer.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: ahah_typo-1567530-6.patch, failed testing.

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.