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

Dropdown: placeholder value #1299

Closed
layershifter opened this issue Feb 8, 2017 · 5 comments
Closed

Dropdown: placeholder value #1299

layershifter opened this issue Feb 8, 2017 · 5 comments
Assignees

Comments

@layershifter
Copy link
Member

Steps

  1. Open codepen.
  2. Click on Dropdown element, it's value will be set to Category 1
  3. Click outside, Dropdown's value will be again Category

Expected Result

Until Dropdown's item will be selected, keep placeholder as Category.

Actual Result

After you click on Dropdown it changes value to first option.

Version

Latest.

Testcase

http://codepen.io/pzurek/pen/mRJMov


@levithomason I'm not sure that this behaviour is bug. Can you clarify?

Based on stackoverflow question.

@levithomason
Copy link
Member

levithomason commented Feb 11, 2017

I believe this is the expected behavior and in line with SUIR core:
http://semantic-ui.com/modules/dropdown#dropdown

http://g.recordit.co/fYXZZvw6KX.gif

The difference we have is that the dropdown is not making Category 1 active on blur, whereas, SUIR core would. We are also changing the placeholder rather than the text. This makes our interaction more jarring than SUIR core.

Proposals, on selected (highlighted) item change:

  • change the text, not the placeholder, for selection
  • change the placeholder only for search Dropdowns
  • do not change text for regular Dropdown menus

(I responded on SO as well)

@layershifter layershifter changed the title question(Dropdown): placeholder value Dropdown: placeholder value Feb 22, 2017
@levithomason
Copy link
Member

We've largely addressed these issues now in the latest version. Feel free to open if you think more needs done.

@layershifter
Copy link
Member Author

@levithomason I reopened this issue because it wasn't fixed and I want to pick up it.

@layershifter layershifter reopened this Aug 28, 2017
@EulerTourist
Copy link

Hi, I'm needing this exact behaviour as well, as described by levithomason. I'd be happy to come on as a test resource.

@stale
Copy link

stale bot commented Feb 4, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 30 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 4, 2018
@stale stale bot closed this as completed Mar 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants