Skip to content

Added autocomplete attribute in the comment form#2739

Closed
thejaydip wants to merge 9 commits intoWordPress:trunkfrom
thejaydip:trunk
Closed

Added autocomplete attribute in the comment form#2739
thejaydip wants to merge 9 commits intoWordPress:trunkfrom
thejaydip:trunk

Conversation

@thejaydip
Copy link
Copy Markdown

Added autocomplete attribute in the comment form - wp-includes/comment-template.php
Ref. (https://core.trac.wordpress.org/ticket/55779)

),
sprintf(
'<input id="author" name="author" type="text" value="%s" size="30" maxlength="245"%s />',
'<input id="author" name="author" type="text" value="%s" size="30" maxlength="245"%s autocomplete="name" />',
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small edit: This has an extra space before maxlength

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@dream-encode
Copy link
Copy Markdown
Contributor

Thanks for the PR! This was merged into core in https://core.trac.wordpress.org/changeset/53576.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants