form should have snippet placeholder #7

Closed
akshatpradhan opened this Issue Apr 26, 2012 · 4 comments

2 participants

@akshatpradhan

This part of the stencil: http://akshatpradhan.dropmark.com/50023/556688

Needs to have a little snippet text below like this: http://akshatpradhan.dropmark.com/50023/556689

@toddmoy

@akshatpradhan thanks for including the screencaps -- but Dropmark is throwing a Permission Denied error. Maybe there's a public setting that needs to be enabled?

@toddmoy toddmoy was assigned Apr 27, 2012
@akshatpradhan

You're right, I had to set it to public.

@akshatpradhan

I was also think, maybe we should have a placeholder text inside the box as well. It's becoming pretty common to use the placeholder attribute. See how "First Name" is greyed out?

@toddmoy toddmoy closed this in 83ee744 May 2, 2012
@toddmoy

I agree about the help text and I've added this update in the latest push.

I stopped short of adding the placeholder text though. In principle, I agree that this is a good affordance. But since I expect many folks will be adding dummy (but not placeholder) content into these fields, I kept the text color black. It's a few extra clicks to set the text in grey, but it seems ok in this case. Willing to reconsider if there's an outpouring of support for the placeholder-esque style.

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