Avoid having to set inline styles to hide modal by default. Fix issue #…

…186
kylefox · Oct 17, 2017 · 48b20e4 · 48b20e4
1 parent a9b77a9
commit 48b20e4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/jquery.modal.css b/jquery.modal.css
@@ -21,7 +21,7 @@
  background-color: transparent;
 }
 .modal {
- display: inline-block;
+ display: none;
  vertical-align: middle;
  position: relative;
  z-index: 2;

diff --git a/jquery.modal.js b/jquery.modal.js
@@ -73,6 +73,7 @@
  }
  } else {
  this.$elm = el;
+ this.anchor = el;
  this.$body.append(this.$elm);
  this.open();
  }
@@ -143,9 +144,9 @@
  }
  this.$elm.addClass(this.options.modalClass).appendTo(this.$blocker);
  if(this.options.doFade) {
- this.$elm.css('opacity',0).show().animate({opacity: 1}, this.options.fadeDuration);
+ this.$elm.css({opacity: 0, display: 'inline-block'}).animate({opacity: 1}, this.options.fadeDuration);
  } else {
- this.$elm.show();
+ this.$elm.css('display', 'inline-block');
  }
  this.$elm.trigger($.modal.OPEN, [this._ctx()]);
  },