Cause: IE7 and IE8 do not have String.prototype.trim function.

Solution:
In metatag.vertical-tabs.js:

        vals.push(Drupal.t('@label: @value', {
          '@label': label.trim(),
          '@value': Drupal.truncate($(this).val(), 25) || Drupal.t('None')
        }));

Should be:

        vals.push(Drupal.t('@label: @value', {
          '@label': $.trim(label),
          '@value': Drupal.truncate($(this).val(), 25) || Drupal.t('None')
        }));

Comments

Anonymous’s picture

Thanks for the solution - It seems this bug breaks CKEditor in IE7 and IE8, but the solution above fixed it.

Is there any reason this hasn't been added to a release yet? Anything I can do to help speed that up?

jtse’s picture

#1, I had the same problem -- TinyMCE broke. I think this is getting overlooked because it's an old IE issue.

dave reid’s picture

Version: 7.x-1.0-alpha3 » 7.x-1.x-dev
Status: Active » Fixed
jtse’s picture

Thanks Dave!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.