I could swear that the email, url, number types all supported the placeholder attribute.

I just noticed they don't. Patch coming soon.

CommentFileSizeAuthor
#2 1330024-placeholder.patch757 bytesboombatower

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Postponed (maintainer needs more info)

Not sure why they wouldn't be right now? 'placeholder' is one of the array values in element_set_attributes() in all the theme functions for those elements.

boombatower’s picture

boombatower commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 1330024-placeholder.patch757 bytes

I'm sure you can add more, I just need this for now.

ericduran’s picture

ericduran commented
Title: Placeholder for email, number, etc » Placeholder for password

Yea, I guess I was just looking at the element_info decleration at the time.

Change the title for the password element

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Great. This has been committed to both 7.x-1.x and 6.x-1.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.