A patch to form.inc a few weeks ago meant that underscores in all form identifiers are now replaced with hyphens. This patch fixes the 'Add a new comment' link so that the page scrolls correctly to the comment submission form.

CommentFileSizeAuthor
comment_form.patch1.13 KBhickory
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

RobRoy’s picture

RobRoy CreditAttribution: RobRoy commented
Status: Needs review » Reviewed & tested by the community

Applies and looks good. Nice catch.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Good job and nice catch indeed. Committed.

alex_b’s picture

alex_b CreditAttribution: alex_b commented

This change caused a white screen on the not up to date version of jscalendar.module (HEAD rev. 1.12). I post this here for reference: http://drupal.org/node/108537 .

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented

Thanks for noting the bug with jscalendar alex_b. The bug turned out on investigation not to be related to this comment-form change.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)