| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
PatternFly-React preview: https://patternfly-react-pr-2820.surge.sh |
Sorry, something went wrong.
There was a problem hiding this comment.
This is looking good, But a couple of questions:
When I add a new item to the list, looks like it's always added at the end. Is this always the case? If we have an alphabetically list, shouldn't the new item be inserted where it belongs? So if I add Connecticut, why does it get added to the end rather than after Alabama?
Shouldn't these options also apply to the multi-value type ahead select?
Sorry, something went wrong.
|
@mcarrano Both new and creatable should work for multi-type ahead select as well. I will add the checkbox toggles to that example! |
Sorry, something went wrong.
There was a problem hiding this comment.
A few comments. Otherwise looks good. Thanks.
Sorry, something went wrong.
Sorry, something went wrong.
|
For the multiple typeahead select input when I have Alabama filtered and then I search for Alabama again and add it, both of them delete and it wipes Alabama clear? I don't know if this is the expected behavior? |
Sorry, something went wrong.
|
When the menu expands up, should the arrow rotate @mcarrano |
Sorry, something went wrong.
|
@jschuler autocomplete is set to off for all selects, so I'm unsure why it would be showing up. Which browser are you using? @christiemolloy isCreatable handles whether the user can add inputs, but will only do so temporarily without onCreateOption defined. onCreateOption is a callback for the user to store the new option in the overall options list, and is used together with isCreateable. Re: multi-typeahead, it looks like you are selecting Alabama twice, first which selects it and second which unselects it and removes it form the chip group. |
Sorry, something went wrong.
|
@kmcfaul strange, Chrome 76.0.3809.132. If no one else sees this then it might be something on my end |
Sorry, something went wrong.
|
That's my version of Chrome and I'm not seeing the autocomplete on the surge. Very odd. Anyone else seeing it? |
Sorry, something went wrong.
Sorry, something went wrong.
|
@kmcfaul do you have autofill enabled? chrome://settings/addresses |
Sorry, something went wrong.
|
I do have it enabled. |
Sorry, something went wrong.
|
Might have to wrap in form tags as this comment suggests https://gist.github.com/niksumeiko/360164708c3b326bd1c8#gistcomment-3004386 |
Sorry, something went wrong.
|
That seems less than ideal, html structure wise. Will try it out though, hopefully shouldn't change anything visually. |
Sorry, something went wrong.
|
@kmcfaul ok, i don't recall seeing autocomplete before, i wonder why it's showing now |
Sorry, something went wrong.
|
Regarding your question @christiemolloy:
I don't feel strongly about this, but I think we should just be consistent with what we do for other dropdown or select instances. |
Sorry, something went wrong.
|
I think it makes sense to have the arrow icon change to an upward direction on direction toggle. This is more of a quick bugfix than related to this particular PR though. I can put up a different PR for it. |
Sorry, something went wrong.
|
@kmcfaul wrapping inputs with form elements fixed the auto complete issue for me |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
What: Adds the creatable and new use cases to select via isCreatable and onCreateOption properties. Specifically applies to typeahead variants.
Refer to issue: #2665