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

Placeholder text does not appear for search bar when expanded #25

Closed
bazhip opened this issue Dec 30, 2016 · 3 comments
Closed

Placeholder text does not appear for search bar when expanded #25

bazhip opened this issue Dec 30, 2016 · 3 comments

Comments

@bazhip
Copy link

bazhip commented Dec 30, 2016

This repository's issues are reserved for feature requests and bug reports.

Do you want to request a feature or report a bug?

bug

Feature Request

please first make sure your request falls under the official Material Design spec guidelines https://material.google.com/

Bug Report

please provide steps to reproduce and if possible screenhots or animated Gifs.
you can easily create animated Gif with this free PC/OSX App: http://www.cockos.com/licecap/

Screenshots or link to CodePen/Plunker/JSfiddle

searchbar

What is the expected behavior?

I'd think the placeholder text should appear when the search is expanded so that they know what the field is for. I ended up adding a block of text to the side for the time being until I play with it more.

What is the motivation / use case for changing the behavior?

^

Which version of Angular and Material, and which browser and OS does this issue affect?

I'm on the latest build (with 2.4.1) and using chrome

Did this work in previous versions of Angular / Material?
Please also test with the latest stable and snapshot versions.
Haven't checked the others

Other information

(e.g. detailed explanation, stacktraces, related issues, suggestions how to fix)

@emoralesb05
Copy link
Collaborator

The problem is we autofocus when you click on the icon and we hide the placeholder when focused.. we might need to add an attribute to make that optional though.

Can you please create this issue or at least link it in https://github.com/Teradata/covalent/issues?

@bazhip
Copy link
Author

bazhip commented Jan 1, 2017

Yeah I deduced that after playing with it for a minute. I just edited it to not focus, but an option would be great to have.

@emoralesb05
Copy link
Collaborator

Sure, we can continue our talk in the covalent issues
Ill close this issue in favor of Teradata/covalent#226

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