Skip to content
This repository has been archived by the owner on Sep 10, 2019. It is now read-only.

Allow modals to receive a controller, resolvers and a few other options. #689

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lunks
Copy link

@lunks lunks commented Sep 10, 2015

This follows pretty much what angular-ui-bootstrap does for attaching controllers and resolve things around.

I've changed the way we fetch templates to use $templateRequest and I'm exposing modalInstance to the controller so we can close it from there if we want to.

I'm also using `$templateRequest` to fetch the template instead of
`$http`.

This follows pretty much what angular-ui-bootstrap does for attaching
controllers and resolve things around.
@zurbchris
Copy link
Contributor

Can you clean this up and only submit changes to relevant individual files? We'll handle creating the /dist files.

@gruberb
Copy link

gruberb commented Apr 4, 2016

What's the status on this?

@lunks
Copy link
Author

lunks commented Apr 4, 2016

I left the previous company I was using foundation-apps and forgot about this. I'll do what @zurbchris asked so we can move forward.

@soumak77
Copy link
Contributor

Feel free to submit this PR against the new fork

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants