From adf836896a39a7ded85686ca6158f3f889d0f384 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=A1bor=20Hojtsy?= <gabor@hojtsy.hu> Date: Tue, 8 Jan 2008 19:54:07 +0000 Subject: [PATCH] #204756 by dvessel: textarea.js assumed #disabled fields are to be hidden --- misc/teaser.js | 11 +++++++++-- misc/textarea.js | 11 ----------- modules/system/system.css | 4 ++++ 3 files changed, 13 insertions(+), 13 deletions(-) diff --git a/misc/teaser.js b/misc/teaser.js index 37e31873de14..633d39e6c70a 100644 --- a/misc/teaser.js +++ b/misc/teaser.js @@ -70,13 +70,20 @@ Drupal.behaviors.teaser = function(context) { body[0].value = trim(text[1]); $(teaser).attr('disabled', ''); $('input', button).val(Drupal.t('Join summary')).toggle(join_teaser, split_teaser); - $(teaser).show(); } else { - $(teaser).hide(); $('input', button).val(Drupal.t('Split summary at cursor')).toggle(split_teaser, join_teaser); $(checkbox).hide(); } + // Make sure that textarea.js has done its magic to ensure proper visibility state. + if (Drupal.behaviors.textarea && teaser.is(('.form-textarea:not(.textarea-processed)'))) { + Drupal.behaviors.textarea(teaser.parentNode); + } + // Set initial visibility + if ($(teaser).is('[@disabled]')) { + $(teaser).parent().hide(); + } + }); }; diff --git a/misc/textarea.js b/misc/textarea.js index fed9db687d2e..f2c9962d1319 100644 --- a/misc/textarea.js +++ b/misc/textarea.js @@ -4,22 +4,11 @@ Drupal.behaviors.textarea = function(context) { $('textarea.resizable:not(.textarea-processed)', context).each(function() { var textarea = $(this).addClass('textarea-processed'), staticOffset = null; - // Make sure that teaser.js has done its magic before converting this textarea. - if (Drupal.behaviors.teaser && textarea.is(('.teaser:not(.teaser-processed)'))) { - Drupal.behaviors.teaser(this.parentNode); - } - // When wrapping the text area, work around an IE margin bug. See: // http://jaspan.com/ie-inherited-margin-bug-form-elements-and-haslayout $(this).wrap('<div class="resizable-textarea"><span></span></div>') .parent().append($('<div class="grippie"></div>').mousedown(startDrag)); - // Inherit visibility - if ($(this).is('[@disabled]')) { - $(this).parent().hide(); - $(this).show(); - } - var grippie = $('div.grippie', $(this).parent())[0]; grippie.style.marginRight = (grippie.offsetWidth - $(this)[0].offsetWidth) +'px'; diff --git a/modules/system/system.css b/modules/system/system.css index 0f906d19f3e9..86e4f32db63e 100644 --- a/modules/system/system.css +++ b/modules/system/system.css @@ -392,6 +392,10 @@ a.tabledrag-handle-hover .handle { background-position: center 1px; margin-bottom: -2px; } +/* Keeps inner content contained in Opera 9. */ +.teaser-checkbox { + padding-top: 1px; +} div.teaser-button-wrapper { float: right; /* LTR */ padding-right: 5%; /* LTR */ -- GitLab