Move to pkg/js – away from js_wrapping

[kevmoo]

Would be super nice 😄

[a14n]

It's something I'm looking at but:

1. I'd like to keep the same public API ( that means wrapping almost every objects - btw once Allow @JS on class method dart-lang/sdk#24779 fix wrapping could be removed )
2. It's not that simple regarding the size of the package

[Andruj]

Right now, this is breaking using google_maps with angular2. Pub will resolve to angular2: 2.0.0-alpha 6 because of the js_wrapping dependency in Google Maps ^3.0.0

[a14n]

@Andruj as I said in #38 using dependency_overrides solves your issue...

[kevmoo]

Can we update js_wrapping ?

[a14n]

@kevmoo I'm working on it :)

[erikist]

Pinging to see what the status on this is: I am having an issue using google_maps and the pwa package.

[a14n]

I'm waiting for dart-lang/sdk#35084 before removing dart:js usage.

@jmesserly let me know if google_maps could be a good candidat to test your work. I'd be happy to help. Could you confirm that dart-lang/sdk#24779 will be fixed (I don't see it referenced in your tracking issue)? I'd like especially to know if the following code will be possible:

@JS('google.maps.Map')
class Map {
  LatLngBounds get bounds => _getBounds();
  @JS('getBounds') LatLngBounds _getBounds();
}

[jmesserly]

Yeah definitely, rename support is critical. Thanks for letting me know about that bug!

[jmesserly]

Oh also, yeah google_maps would be a great test case!

js_wrapping is a great source of inspiration too, by the way. I hope the new compiler support will have quite a similar feel, from a usability perspective. (And the compilers should be able to avoid wrappers, generally speaking.)

[a14n]

Version 4.0.0 now uses package:js instead of dart:js.