Minor tweaks to the custom modal sample to remove dependencies on jQu…

…ery and make things more angular-ish
dwmkerr · Oct 19, 2014 · d73313b · d73313b
1 parent cd8c9bc
commit d73313b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/samples/custom/custom.html b/samples/custom/custom.html
@@ -1,6 +1,5 @@
 <style type="text/css">
 #overlay {
-    display: none;
     position: fixed;
     left: 25%;
     top: 25%;
@@ -12,7 +11,6 @@
     z-index: 100;
 }
 #fade {
-    display: none;
     position: fixed;
     left: 0%;
     top: 0%;
@@ -25,9 +23,9 @@
     z-index: 90;
 }
 </style>
-<div id="overlay">
+<div id="overlay" ng-show="display">
     This is a custom modal. The angular-modal-service doesn't depend on bootstrap, you
     can use any modal you want.
     <a href ng-click="close()">Close</a>
 </div>
-<div id="fade"></div>
+<div id="fade" ng-show="display"></div>
diff --git a/samples/custom/customcontroller.js b/samples/custom/customcontroller.js
@@ -2,7 +2,10 @@ var app = angular.module('sampleapp');
 
 app.controller('CustomController', ['$scope', 'close', function($scope, close) {
 
+  $scope.display = true;
+
   $scope.close = function() {
+    $scope.display = false;
  	  close();
  };
 

diff --git a/samples/sampleapp.js b/samples/sampleapp.js
@@ -44,7 +44,6 @@ app.controller('SampleController', ['$scope', 'ModalService', function($scope, M
       templateUrl: "custom/custom.html",
       controller: "CustomController"
     }).then(function(modal) {
-      modal.element.css("display", "block");
       modal.close.then(function(result) {
         $scope.customResult = "All good!";
       });