There is a @todo in the recently committed CKeditor implementation, which causes the $config['block_formats'] value to be ignored:

  if (isset($config['block_formats'])) {
    // @todo This is now format_tags and needs format_[tag] settings per tag.
    //$settings['FontFormats'] = strtr($config['block_formats'], array(',' => ';'));
  }

Here is the patch.

In fact, it already seemed to work when I only replaced $settings['FontFormats'] with $settings['format_tags'], but I still added the foreach-part because the CKeditor docs say:

Each entry must have its relative definition configuration in a setting named "format_(tagName)".
CommentFileSizeAuthor
ckeditor_inc_format_tags.patch807 bytesmarcvangend

Comments

twod’s picture

Version: 6.x-2.x-dev » 7.x-2.x-dev
Category: task » feature

Oh, I thought the class property was required for that to work, which is probably why I assumed we'd have to leave this for the Advanced settings issue. Should have looked at the actual code instead of the examples...
I'll take a closer look at this later, but the patch itself looks good. (I don't remember if the coding standards say anything about the uppercase 'AS' in loops.)

twod’s picture

Component: Code » Editor - CKeditor

Changing to the new CKEditor component. This might be merged into #689218: Improved support for CKEditor.

sun’s picture

Status: Needs review » Closed (duplicate)

Let's continue over there.