search

fmoo / react-typeahead

Pure react-based typeahead and typeahead-tokenizer
ISC License
677 stars 232 forks source link

Allowing custom tokenizer token values #168

Closed minyoung closed 8 years ago

minyoung commented 8 years ago

Typeahead has the formInputOption property, which can be used to transform a complex option to a form input value. It would be nice to have the same thing for Tokenizer.

minyoung commented 8 years ago

Defaulting to Tokenizer.displayOption seems like a reasonable thing to do. Also updated README.md to include the new Tokenizer.formInputOption

fmoo commented 8 years ago

Thank you SO much for cleaning up this code, also! This is great!

fmoo commented 8 years ago

Might need to do some post processing on the default if the display value is an HTMLElement instead of text, but we can burn that bridge when we get to it.