mbirth/LuckyCoinkydink
1
0
Fork 0
Code Issues Pull Requests Activity

A better solution for media file/template info overlays

Browse Source 
Since the JS-based solution does not work well for cases where the
overlay is larger than the (very small) image, we'll have to go
with a CSS-based solution. This sets the height of the overlay
to at least 7em, which should cover the core info of the overlay
(which is usually 4 lines of text). Also, we'll have to go with
overflow: scroll; in case the info is longer.

Not a good solution in a reponsive context, but it should work. :(

References #230
This commit is contained in:
Matthias Mees 2015-01-16 12:27:22 +01:00
parent bdea7336ee
commit 0f2f437c8c
2 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
templates/2k11/admin/serendipity_editor.js.tpl
View File

@ -765,12 +765,6 @@
updateOnResize: true updateOnResize: true
}); });
} }
// Also sync height of info overlays
$('.equal_heights').each(function() {
var $el = $(this);
var containerHeight = $el.css('min-height');
$el.children('.media_file_preview, .template_preview').css('minHeight', containerHeight);
});
} }
} }

3
templates/2k11/admin/style.css
View File

@ -2259,7 +2259,8 @@ label .perm_name,
box-sizing: border-box; box-sizing: border-box;
color: #fcfcfc; color: #fcfcfc;
height: 100%; height: 100%;
overflow: hidden; min-height: 7em;
overflow: scroll;
padding: .5em 1em; padding: .5em 1em;
position: absolute; position: absolute;
top: 0; top: 0;