Merge pull request #215 from jaguadoromero/fix_issue_#186

Avoid having to set inline styles to hide modal by default. Fix #186
kylefox · Oct 18, 2017 · 6b835ec · 6b835ec · jserrao · Oct 18, 2017
2 parents 23074d7 + 48b20e4
commit 6b835ec
Show file tree

Hide file tree

Showing 2 changed files with 3 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
@@ -144,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()]);
  },