From 0f2f437c8c4e7d99b2aa3b4fd20ec107b2c63d65 Mon Sep 17 00:00:00 2001 From: Matthias Mees Date: Fri, 16 Jan 2015 12:27:22 +0100 Subject: [PATCH] A better solution for media file/template info overlays 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 --- templates/2k11/admin/serendipity_editor.js.tpl | 6 ------ templates/2k11/admin/style.css | 3 ++- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/templates/2k11/admin/serendipity_editor.js.tpl b/templates/2k11/admin/serendipity_editor.js.tpl index dde1304e..6cb07c6f 100644 --- a/templates/2k11/admin/serendipity_editor.js.tpl +++ b/templates/2k11/admin/serendipity_editor.js.tpl @@ -765,12 +765,6 @@ 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); - }); } } diff --git a/templates/2k11/admin/style.css b/templates/2k11/admin/style.css index a1271166..3d72a6c5 100644 --- a/templates/2k11/admin/style.css +++ b/templates/2k11/admin/style.css @@ -2259,7 +2259,8 @@ label .perm_name, box-sizing: border-box; color: #fcfcfc; height: 100%; - overflow: hidden; + min-height: 7em; + overflow: scroll; padding: .5em 1em; position: absolute; top: 0;