angular adapter

[yoshuawuyts]

@bendrucker @lrlna I honestly don't know who else to ask lol, if you could help out writing an adapter I'd owe you one haha ✨

[lrlna]

can totes help out with this one ✨

[lrlna]

Oh actually, which angular? I can help you with pre-2.0 directive; haven't done 2.0 stuff.

…

On Mon, Jan 30, 2017, 21:03 Yoshua Wuyts ***@***.***> wrote: @bendrucker <https://github.com/bendrucker> @lrlna <https://github.com/lrlna> I honestly don't know who else to ask lol, if you could help out writing an adapter I'd owe you one haha ✨ — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHu3CA3T9uZVVqGsSMaxYXRMd8f_QueNks5rXlAxgaJpZM4Lx6VK> .

[bendrucker]

Same, haven't touched v2

[yoshuawuyts]

Angular v1 stuff would be grand I think; if I'm not mistaken that seems to be the thing that's used most nowadays anyway / what people end up working with - so if we can make their lives easier would be grand :D

…

On Mon, Jan 30, 2017 at 1:38 PM Ben Drucker ***@***.***> wrote: Same, haven't touched v2 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWlei4xSnVlUPgXrPE6k8UmLGkbbkkLks5rXlhugaJpZM4Lx6VK> .

[bendrucker]

Cool. Shouldn't be too hard. Any thoughts on unit testing this stuff?

[yoshuawuyts]

ohh, yeah unit tests would be neat - didn't write any so far, but I bet using `tape-run` we could come a long way; probably should also add some to `nanocomponent` haha

…

On Mon, Jan 30, 2017 at 3:22 PM Ben Drucker ***@***.***> wrote: Cool. Shouldn't be too hard. Any thoughts on unit testing this stuff? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWlesHrdzkeRWh4foiZ-XNnnRA5hJFrks5rXnCYgaJpZM4Lx6VK> .

[lrlna]

uhhhh i haven't componentized angular in so long, so I am not sure if this is the right approach atm. Something like this for the spec?

var nanoToAngular = require('nanocomponent-adapters/angular')

module.exports = angular
  .module('nano', [])
  .directive('nanocomponent', nanoToAngular)

and hook it up in your html:

<nanocomponent render></nanocomponent>
<nanocomponent onupdate></nanocomponent>

So then component vals get handled in html. That will mean that the directive itself will have to handle all the nanocomponent interaction too.

Alternatively handle it within the $scope instead of working inside html directly.

[bendrucker]

That looks right to me!

[alterx]

Hey @lrlna, @bendrucker, @yoshuawuyts, may I suggest the following naming convention:
nanocomponent-adapters/angularjs for Angular 1.x
nanocomponent-adapters/angular for Angular 2+

Recently there's been a lot of discussion around this topic and the Angular team is settling with "just Angular" for the newer versions of the framework and AngularJS for 1.x
See "Naming guidelines" section

Also, I've been doing some Angular work so I'd be happy to port the AngularJS version.

[yoshuawuyts]

If that's the convention, it sounds v reasonable to me. Afaik nobody's offered to write the Angular (2.x+) version yet so help would def be welcome! On Sun, Feb 12, 2017, 18:27 Carlos Vega <[email protected]> wrote: Hey @lrlna <https://github.com/lrlna>, @bendrucker <https://github.com/bendrucker>, @yoshuawuyts <https://github.com/yoshuawuyts>, may I suggest the following naming convention: nanocomponent-adapters/angularjs for Angular 1.x nanocomponent-adapters/angular for Angular 2+ Recently there's been a lot of discussion around this topic and the Angular team is settling with "just Angular" for the newer versions of the framework and AngularJS for 1.x See "Naming guidelines" section <http://angularjs.blogspot.com/2016/12/ok-let-me-explain-its-going-to-be.html> Also, I've been doing some Angular work so I'd be happy to port the AngularJS version. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWlev6milx4yaG-I4ZMYyTGMlhayGTFks5rb0EIgaJpZM4Lx6VK> .

[alterx]

fyi, PR #6 adds support for custom elements and, consequently, for Angular (2+)

[davidmarkclements]

from angular 1.5 there's also the component method - it's just sugar on top of a directive but it would be cool if there were unit tests passing for both directive and component