You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Here are a few things I'm noticing while using this library. This issue is here for discussion and any items I can take and raise a PR for.
autoCreate by itself does not always ensure the search box is visible. I found you need to use displaySearchStatus=always. This is something I missed (the example documentation worked without this but in my app, it does not).
updateEvent is triggering when autoCreateItem is emitted. I think it should be suppressed for this case. I need feedback here as remove option will emit an update, but for adding a new one, the value is of a string and not the underlying item.
Search box could benefit from Placeholder to help the end user know. If no autocreate, then "Filter", else "Filter or create new item". This could help a ton for end users. Ideally we'd expose this as an input.
autoCreate on adding can leave some empty state in the search box which doesn't show the remaining options unless search box is cleared out (this only happens in my app, not in example page)
autoCreate mode might want a button to indicate to the user they can add (or an option that's not selectable to tell them they can add"
The text was updated successfully, but these errors were encountered:
If the proposal is coherent and does not break the existing, you can propose an evolution.
This is not a feature I personally need for the projects I work on.
The component is already a fork of another project, I haven't really taken the time to review certain points that could clearly be improved.
It's true that it's not super consistent with the documentation. I forgot this point.
Perhaps the sending should be made more consistent indeed.
Perhaps going up in the "selection" rather than the "dropdown" will be more consistent.
Here are a few things I'm noticing while using this library. This issue is here for discussion and any items I can take and raise a PR for.
autoCreate
by itself does not always ensure the search box is visible. I found you need to usedisplaySearchStatus=always
. This is something I missed (the example documentation worked without this but in my app, it does not).autoCreate
on adding can leave some empty state in the search box which doesn't show the remaining options unless search box is cleared out (this only happens in my app, not in example page)autoCreate
mode might want a button to indicate to the user they can add (or an option that's not selectable to tell them they can add"The text was updated successfully, but these errors were encountered: