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)".
| Comment | File | Size | Author |
|---|---|---|---|
| ckeditor_inc_format_tags.patch | 807 bytes | marcvangend |
Comments
Comment #1
twodOh, 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.)
Comment #2
twodChanging to the new CKEditor component. This might be merged into #689218: Improved support for CKEditor.
Comment #3
sunLet's continue over there.