Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lugg 428 ckeditor further upgrade #7

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

juampynr
Copy link

  • Removes ckeditor module and instead makes it a dependency as its customizations have been moved to either custom code or exported config at luggage_ckeditor.
  • Upgrades ckeditor library and plugins to latest version (4.5.1).
  • Caption support has been added, but an existing bug makes that Firefox users that want to edit a caption need to click at the start of the text and then when the cursor appears, click Shift + down arrow in order to edit the text.
  • SpellAsYouType plugin has been disabled due to another bug.
  • The image button has been removed from the toolbar via JavaScript.
  • The URL field in the image dialog has been removed via JavaScript.

Installation instructions

In a new luggage installation. Open luggage_ckeditor/README.md and follow instructions.

juampynr and others added 7 commits June 18, 2015 13:23
* Upgraded to version 4.4.7 with Image2 plugin.
* Added CKEditor library to the module since it can't
  be downloaded along with the Image2 plugin from the web.
* Updated installation instructions so the ckeditor library
  is copied into sites/all/libraries.
* Adjusted wysiwyg filter rules so captions are supported.
All adjustments have been moved to luggage_ckeditor.
@juampynr juampynr closed this Jul 17, 2015
@juampynr juampynr deleted the LUGG-428-ckeditor-further-upgrade branch July 17, 2015 17:41
@juampynr juampynr restored the LUGG-428-ckeditor-further-upgrade branch July 17, 2015 17:41
@juampynr juampynr reopened this Jul 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants