The CKEditor demo on the CKEditor website shows two language direction buttons on the toolbar as shown in the attached image.

Is it possible to enable these items somehow when using ckeditor with the wysiwyg plugin?

CommentFileSizeAuthor
#5 wysiwyg.ckeditor.bidir_.1112212.5.patch1.76 KBtwod
#3 wysiwyg.master.ckeditor-bidi.1112212.3.patch1.19 KBtwod
Screen shot 2011-03-31 at 17.47.31.png4.13 KBtimdiacon

Comments

timdiacon’s picture

timdiacon commented
Category: support » feature

Right I've worked out how to do this, you need to add the code below to line 321 of ckeditor.inc (function name wysiwyg_ckeditor_plugins)

'BidiLtr' => t('Lang LTR;'), 'BidiRtl' => t('Lang RTL'),

It would be great to have this update included in the next release so I'm going to amend category to feature request

ethnovode’s picture

ethnovode commented

The buttons for ltr/rtl are available in tinymce but not in ckeditor. Would be nice to have.
Thanks timdiacon.

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Version: 6.x-2.3 » 7.x-2.x-dev
Component: Plugins » Editor - CKEditor
Status: Active » Needs review
StatusFileSize
new wysiwyg.master.ckeditor-bidi.1112212.3.patch1.19 KB

Oops, these buttons were overlooked...

Looks like they were added in CKEditor 3.4b build 5808, see ticket #5909 and the changelog.

I tested the patch with releases 3.3.2, 3.4b and 3.6 to make sure the buttons are only available for the relevant versions and have not been removed recently.

EDIT: We normally create patches against the master branch, but this one should apply to all.

prabhatjn’s picture

prabhatjn commented
Status: Needs review » Reviewed & tested by the community

This patch works!!
Is it already committed?

Perry.

twod’s picture

twod
Primary language Swedish
Location Sweden
commented
Status: Reviewed & tested by the community » Fixed
StatusFileSize
new wysiwyg.ckeditor.bidir_.1112212.5.patch1.76 KB

Fixed the incorrect button name in in the unset() and committed this to all branches.

The -dev snahpshots will be updated within 12hrs and this patch will be in the next release.
Thanks for reporting, patching, testing and commenting!

Status: Fixed » Closed (fixed)

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

hefox’s picture

hefox commented
Version: 7.x-2.x-dev » 6.x-2.x-dev
Status: Closed (fixed) » Needs review

Any chance of backport?

Patch seems to work fine so far.