Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Semantic Placeholder for Search Input #9

Open
jpdevries opened this issue Oct 19, 2016 · 5 comments
Open

Semantic Placeholder for Search Input #9

jpdevries opened this issue Oct 19, 2016 · 5 comments

Comments

@jpdevries
Copy link
Contributor

Took a swing at this but didn't get it to work design wise. <input placeholder> is meant to display an example of a valid value for the input like "Content Blocks". It is not meant to describe the input or prompt the user.

@Mark-H
Copy link
Member

Mark-H commented Oct 19, 2016

Looks like you did sorta get this done? Not sure if "Content Blocks" is a great placeholder though, that doesn't make much sense as a search term for 90% of the pages.

@jpdevries
Copy link
Contributor Author

Oh I didn't mean to commit that change to the placeholder 

Get Outlook for iOS

On Thu, Oct 20, 2016 at 12:12 AM +0200, "Mark Hamstra" [email protected] wrote:

Looks like you did sorta get this done? Not sure if "Content Blocks" is a great placeholder though, that doesn't make much sense as a search term for 90% of the pages.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@Mark-H
Copy link
Member

Mark-H commented Nov 1, 2016

I've set the placeholder back to the "Search documentation" for now. There's still the hidden label you added, so is that sufficient do you think?

@jpdevries
Copy link
Contributor Author

Technically no. The placeholder is meant to suggest a valid entry like "extras" or even just "documentation" but it is not meant to describe the input itself.

Not very high priority, but now it might be confusing and will read 'search documentation' twice to screen readers.

@jpdevries
Copy link
Contributor Author

More on the float label pattern http://bradfrost.com/blog/post/float-label-pattern/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants