Removed useless resize() function, now using CSS.

Author: zewish

bower.json

@@ -1,7 +1,7 @@
 {
   "name": "rmodal",
   "main": "rmodal.js",
-  "version": "1.0.9",
+  "version": "1.0.10",
   "authors": [
     "Iskren Slavov <iskren.s@gmail.com>"
   ],

dist/rmodal.css

@@ -1,5 +1,4 @@
 .modal {
-    position: absolute;
     display: none;
     background: rgba(0, 0, 0, .30);
     z-index: 999;
@@ -26,6 +25,15 @@ body.no-bootstrap.modal-open {
 
 body.no-bootstrap .modal {
     padding: 30px 0;
+
+    position: fixed;
+    top: 0;
+    left: 0;
+    right: 0;
+    bottom: 0;
+
+    overflow-x: hidden;
+    overflow-y: auto;
 }
 
 body.no-bootstrap .modal .modal-dialog {

dist/rmodal.min.js

@@ -43,10 +43,6 @@
                 // , escapeClose: true
             });
 
-            window.addEventListener('resize', function() {
-                modal.resize();
-            }, false);
-
             document.addEventListener('keydown', function(ev) {
                 modal.keydown(ev);
             }, false);

index.html

@@ -1,6 +1,6 @@
 {
   "name": "rmodal",
-  "version": "1.0.9",
+  "version": "1.0.10",
   "description": "A simple modal dialog with no external dependencies. IE9+ supported.",
   "main": "angular-messenger.js",
   "repository": {

package.json

@@ -1,5 +1,4 @@
 .modal {
-    position: absolute;
     display: none;
     background: rgba(0, 0, 0, .30);
     z-index: 999;
@@ -26,6 +25,15 @@ body.no-bootstrap.modal-open {
 
 body.no-bootstrap .modal {
     padding: 30px 0;
+
+    position: fixed;
+    top: 0;
+    left: 0;
+    right: 0;
+    bottom: 0;
+
+    overflow-x: hidden;
+    overflow-y: auto;
 }
 
 body.no-bootstrap .modal .modal-dialog {

src/rmodal.css

@@ -79,7 +79,6 @@
         addClass(this.dialog, this.options.dialogOpenClass);
 
         this.overlay.style.display = 'block';
-        this.resize();
 
         if (this.options.focus) {
             this.focusOutElement = document.activeElement;
@@ -131,18 +130,6 @@
         this.dialog.innerHTML = content;
     };
 
-    RModal.prototype.resize = function() {
-        var overlayWidth = window.innerWidth;
-        var overlayHeight = window.innerHeight;
-        if (document.body.clientHeight > window.innerHeight) {
-            overlayWidth = document.body.clientWidth;
-            overlayHeight = document.body.clientHeight;
-        }
-
-        this.overlay.style.width = overlayWidth + 'px';
-        this.overlay.style.height = overlayHeight + 'px';
-    };
-
     RModal.prototype.elements = function(selector, fallback) {
         fallback = fallback || (window.navigator.appVersion.indexOf('MSIE 9.0') > -1);
         return [].filter.call(this._elementsAll(selector), function(element) {

src/rmodal.js

@@ -215,15 +215,6 @@ describe('RModal', function() {
             expect(instance.overlay.style.display).to.be.equal('block');
         });
 
-        it('should call "this.resize()"', function() {
-            var stub = sinon.stub(RModal.prototype, 'resize');
-            var instance = create();
-
-            instance._doOpen();
-            expect(stub.calledOnce).to.be.true;
-            RModal.prototype.resize.restore();
-        });
-
         it('should set "this.focusOutElement" to "document.activeElement"', function() {
             var expected = document.activeElement;
             var instance = create();
@@ -395,40 +386,6 @@ describe('RModal', function() {
         });
     });
 
-    describe('resize()', function() {
-        it('should set "this.overlay.style[width,height]" to "window[innerWidth,innerHeight]"'
-            , function() {
-                var instance = create();
-
-                elBody.clientHeight = 312;
-                elBody.clientWidth = 422;
-
-                window.innerHeight = 531;
-                window.innerWidth = 542;
-
-                instance.resize();
-
-                expect(instance.overlay.style.width).to.be.equal(window.innerWidth + 'px');
-                expect(instance.overlay.style.height).to.be.equal(window.innerHeight + 'px');
-            }
-        );
-
-        it('should set "this.overlay.style[width,height]" to "body[clientWidth,clientHeight]"'
-            , function() {
-                var instance = create();
-
-                elDialog.innerHTML = '<br/><br/><br/><br/><br/><br/><br/><br/><br/><br/>';
-                window.innerHeight = 123;
-                window.innerWidth = 182;
-
-                instance.resize();
-
-                expect(instance.overlay.style.width).to.be.equal(elBody.clientWidth + 'px');
-                expect(instance.overlay.style.height).to.be.equal(elBody.clientHeight + 'px');
-            }
-        );
-    });
-
     describe('elements()', function() {
         it('should call "this._elementsAll" passing the "selector" param', function () {
             var spy = sinon.spy(RModal.prototype, '_elementsAll');