-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Reuse ui.bootstrap.typeahead functionality #1
Comments
@dimirc @ProLoser I saw your previous discussion so jumping here. What I'm planning to do in the near (well, 3-6 weeks) future is to extract typeahead's functionality into a separate controller to make it extensible. So if we need to sync on refactorings just ping me. Not sure how close those 2 directives are and how much code we can reuse, though. |
There are some interesting comments on this topic at angular-ui/bootstrap#454 |
@pkozlowski-opensource you mean a new repository with just typeahead (without additional dependencies) or what's the idea? |
No, I was refferring to internal refactoring of an existing directive. |
@pkozlowski-opensource have you thought about the fact that BS3 drops typeahead and relocates it to a separate project. I was wondering if you gave it thought at all about doing the same. My guess as to why they did it was because they were starting to realize the plugin itself is a project in of itself (compared to almost all the other plugins). |
Although I suppose since you guys have datepickers and timepickers as part of the 'core' then there isn't a big point to arbitrarily leave out typeahead. |
@dimirc FYI I actually didn't start off with their lib because I wanted to start from scratch and see if we could simplify their implementation further and still provide better feature support. I've been working on this sort of lib for the entire lifespan of AngularUI. See #3 If we build this tool to be extensible enough, then you should be able to have the output be the equivalent to Bootstrap HTML and as such it should be ready-to-go. We (or ui-bootstrap) can even provide optional templates or a default configuration of this plugin. However, I want to aim for wider support than strictly bootstrap. |
bare minumum to work with Bootstrap 4
Upgrading Karma
typeahead.js already is an AngularJS native directive, so maybe we could reuse all that functionality and extend it.
typeahead module (ui.bootstrap.typeahead) depends on:
The text was updated successfully, but these errors were encountered: