-
Notifications
You must be signed in to change notification settings - Fork 169
ng-controller doesn't suggest my controller name #67
Comments
This is due to how the files are parsed. To reduce false positives on matching directives/controllers/etc the plugin looks for angular/app/etc so if you're naming your app something other than "app" you'll need to update the regex in the settings file. https://github.com/angular-ui/AngularJS-sublime-package#indexing-options In your case you'd need to change |
Thanks. I added |
Didn't notice that in the screen shot. It's possible, I'll have to look into that. |
I'm actually not so sure I used the right regex, could you clarify? |
@gkatsanos, yeah, looks like you missed updating the This:
should be this:
And you'll need to restart ST before doing the reindexing. |
is there a way to use a wildcard of some type so that all *App names are picked-up? |
Sure, if you'd like. It's converted down into a regex. So anything you can do via Regex in Python is doable there. |
I can't really do Regexes :) Wouldn't it be helpful to make it by default catch all names that end on "app" ? |
@gkatsanos sounds reasonable. I've been considering always going with the second method and exposing an array that you can tweak so it's easier to change the behavior. This should be a pretty safe update so I'll try to hammer something out this week and push an update out. |
Listen if you don't have time I can try to make a pull request myself, I have some time following weekend. |
That'd be very welcomed. I'll still try and find some time this week, but if you can get to it'd be appreciated. |
Hi guys, any update on this? |
I've been slammed with work, per usual, but the adjustments to the config shown above should allow you to make any tweaks needed until updated to be more user friendly. I can't say when, but I'm really wanting to get a day or two set aside to fix reported issues, and produce a new release... |
TODO: add to docs about this update Fixes: #67
I can't get this to work. Here's my app definition and here's my user settings { // {0} is the location of where the definition name will be inserted
} When I type The status message says no available completion.
|
1 similar comment
I can't get this to work. Here's my app definition and here's my user settings { // {0} is the location of where the definition name will be inserted
} When I type The status message says no available completion.
|
Hello.
I have autocomplete enabled, I index my project, but I only get as suggestions the controllers declared inside the angular core libraries, I don't get the one I just wrote:
The text was updated successfully, but these errors were encountered: