angular.dart.ui icon indicating copy to clipboard operation
angular.dart.ui copied to clipboard

Check Carousel unit tests

Open akserg opened this issue 10 years ago • 3 comments

Check what's going wrong with Carousel unit tests environment.

akserg avatar May 27 '14 20:05 akserg

So far, I've worked out that you put css files in index.html because TranscludingComponentFactory doesn't work in this (whereas it did work in ShadowDomComponentFactory). So does this mean that you will have to include all component css files in index.html for all instances of using angular.dart.ui?

Going deeper into 3rd test now.

MadcapJake avatar May 28 '14 01:05 MadcapJake

3rd test succeeds after I pulled your commits, not sure what I did wrong following along with diff.

4th test is failing because you manually listed slide entries instead of using ng-repeat. I'm gonna see if I can work out why ng-repeat is failing.

MadcapJake avatar May 28 '14 01:05 MadcapJake

Hi Jack,

So does this mean that you will have to include all component css files in

index.html for all instances of using angular.dart.ui?

I think all components must use original styles from bootstrap.css. As an exclusion they could have own css files being included on html page. We must avoid workaround to download css files of component ourself. Better use browser to download them, else this way can be fragile and buggy.

Any thoughts, objections?

Regards, Sergey.

On 28 May 2014 03:57, Jake Russo [email protected] wrote:

3rd test succeeds after I pulled your commits, not sure what I did wrong following along with diff.

4th test is failing because you manually listed slide entries instead of using ng-repeat. I'm gonna see if I can work out why ng-repeat is failing.

— Reply to this email directly or view it on GitHubhttps://github.com/akserg/angular.dart.ui/issues/85#issuecomment-44357098 .

akserg avatar May 28 '14 08:05 akserg