First of all, terrific job to the CKEditor team! What a fantastic improvement to an already great editor!

Forcing the simple toolbar for patterns set in CKeditor admin (Force simplified toolbar on the following fields) isn't working for me. In ckeditor.module, function "ckeditor_process_textarea", in order for admin "force simple toolbar" settings to work,

wouldn't line 593

$settings[$textarea_id]['toolbar'] = $conf['toolbar'];

need to be ...

$settings[$textarea_id]['toolbar'] = $toolbar;

That fixed problem for me (my apologies for not submitting as a proper patch format, I'm fairly new to Drupal and not quite up to speed on all the bug/issue/patch protocols).

CommentFileSizeAuthor
#1 ckeditor-652570.patch1 KBmephir
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mephir’s picture

mephir CreditAttribution: mephir commented
Assigned: Unassigned » mephir
Status: Active » Fixed
FileSize
ckeditor-652570.patch1 KB

Patch attached, thanks for information. How to generate properly patch, you can read on http://drupal.org/node/606034/cvs-instructions/HEAD.

WorldFallz’s picture

WorldFallz CreditAttribution: WorldFallz commented
Status: Fixed » Needs review

I don't see this committed or reviewed by the maintainers-- how can be it fixed?

Also, I can confirm this fixes the problem.

wwalc’s picture

wwalc CreditAttribution: wwalc commented
Status: Needs review » Fixed

Thanks WorldFallz for checking it. Fixed in CVS (the updated code should be available in a dev release in 12h).

@mephir - check Status settings for an issue.

Status: Fixed » Closed (fixed)

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