Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix(select): Select option with a label of 0 is not shown. Closes #1401. #1480

Closed
wants to merge 1 commit into from
Closed

Conversation

sudhirj
Copy link
Contributor

@sudhirj sudhirj commented Oct 21, 2012

Bug caused by the use of the || operator to replace all non-truthy values with an empty string. Changed to replace only undefined values.

Closes #1401.

Bug caused by the use of the `||` operator to replace all non-truthy values with an empty string. Changed to replace only `undefined` values.

Closes #1401.
@IgorMinar
Copy link
Contributor

landed as b3cae4f

thanks!

if you don't have our t-shirt yet, please fill out this form: http://goo.gl/075Sj

@IgorMinar IgorMinar closed this Oct 31, 2012
@sudhirj
Copy link
Contributor Author

sudhirj commented Nov 1, 2012

Awesome... had no idea there were t-shirts :D

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

Successfully merging this pull request may close these issues.

ngOptions doesn't render the label 0 (zero)
2 participants