Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check Carousel unit tests #85

Open
akserg opened this issue May 27, 2014 · 3 comments
Open

Check Carousel unit tests #85

akserg opened this issue May 27, 2014 · 3 comments
Assignees
Milestone

Comments

@akserg
Copy link
Owner

akserg commented May 27, 2014

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

@akserg akserg added this to the 0.5 milestone May 27, 2014
@MadcapJake
Copy link

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
Copy link

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.

@akserg
Copy link
Owner Author

akserg commented May 28, 2014

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//issues/85#issuecomment-44357098
.

@akserg akserg modified the milestones: 0.6, 0.5 Jun 5, 2014
@akserg akserg self-assigned this Aug 19, 2014
@akserg akserg modified the milestones: 0.6, 0.7 Oct 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants