bootstrapping and angular docs about angular-hint HOT 17 CLOSED

we determined that it may be trying to bootstrap twice

from angular-hint.

I'm still uncertain about what is going on here. Our call to resumeBootstrap seems to be what causes the thrown error (although the docs site still appears broken whether or not this error is thrown). Specifically, when the module ng tries to instantiate ngLocale the runBlocks for ngLocale is empty and causes an error.

from angular-hint.

I don't think that we are bootstrapping twice. I think that the bootstrapping of ngLocale is failing.

from angular-hint.

Okay... I tried removing all of our hint modules to see if one of them is causing the error, and I still see it triggered. So it would have to be an error in the overall AngularHint logic.

from angular-hint.

I don't know very much about the angular-loader, but I think that it, rather than the AngularHint logic is causing this particular error. For me, building a hint-bundle with just the injector logic calls doBootstrap and attempts to instantiate the empty ngLocale that cause the failure. @btford any insights with regard to the behavior of the loader?

from angular-hint.

I was wondering if I could reproduce this behavior by looking at other sites built with Angular. Both https://builtwith.angularjs.org/ and https://www.virginamerica.com/ are broken by the use of AngularHint, although all the module messages appear to work the same. @caguillen214, it seems like our first priority would be to look at these. Especially https://www.virginamerica.com/ seems to be a good example of how AngularHint could be used with a real application. However, both these sites do not have the same issue as the Angulardocs.

from angular-hint.

The https://www.virginamerica.com/ site is broken with the use of ngHintDom and ngHintInterpolation.

from angular-hint.

The builtwithangular site is broken with the use of ngHintDom, ngHintEvents and ngHintInterpolation.

from angular-hint.

i'm wondering if it's angular-loader + ngLocale that's giving us problems.

from angular-hint.

For the docs site, definitely.

from angular-hint.

I'd focus on the issues you identified, and we can return to the ngLocale thing tomorrow when I'm back.

from angular-hint.

Ok, our plan will to be try to fix individual modules to get a solid demo going for one of these other sites.

from angular-hint.

👍

from angular-hint.

Update: the angular docs site was broken by using an outdated version of Angular loader. However, using the updated Angular loader (1.3) was incompatible with sites running Angular 1.2. Hence we created our own modified loader to work with both versions of Angular, see angular/batarang@e5e6677

from angular-hint.

The https://www.virginamerica.com/ site is broken with the use of ngHintDom and ngHintInterpolation.

ngHintDom is unfeasible except for the patching of action taking place directly within the controller in the current implementation. An alternative solution is being explored.

Issues with ngHintInterpolation were addressed and the module no longer breaks the virginamerica site.

from angular-hint.

The builtwithangular site is broken with the use of ngHintDom, ngHintEvents and ngHintInterpolation

Again, ngHintDom is being revisited, the problem with ngHintEvents is that older builds of Angular have different events, and ngHintInterpolation no longer stops the site from loading correctly (although it does cause harmless exceptions to be thrown - currently being investigated).

from angular-hint.

Closing since the angularjs.org issue is tracked in #88.

from angular-hint.