Browse Source

Fix regression, causing dimmer not to cover whole page when on mobile or scrolling dimmer content.

pull/1063/head
jlukic 10 years ago
parent
commit
f6b015564e
3 changed files with 6 additions and 2 deletions
  1. 2
      src/definitions/modules/dimmer.js
  2. 2
      src/definitions/modules/modal.js
  3. 4
      src/definitions/modules/modal.less

2
src/definitions/modules/dimmer.js

@ -85,7 +85,7 @@ $.fn.dimmer = function(parameters) {
.on(clickEvent + eventNamespace, module.toggle)
;
}
console.log($module, module.is.page());
if( module.is.page() ) {
module.debug('Setting as a page dimmer', $dimmable);
module.set.pageDimmer();

2
src/definitions/modules/modal.js

@ -436,7 +436,7 @@ $.fn.modal = function(parameters) {
cacheSizes: function() {
module.cache = {
pageHeight : $body.outerHeight(),
pageHeight : $(document).outerHeight(),
height : $module.outerHeight() + settings.offset,
contextHeight : (settings.context == 'body')
? $(window).height()

4
src/definitions/modules/modal.less

@ -241,6 +241,10 @@
*******************************/
/* A modal that cannot fit on the page */
body.ui.scrolling.dimmable {
height: auto;
}
.ui.modal.scrolling {
position: absolute;
margin-top: @scrollingTopMargin;

Loading…
Cancel
Save