In #1890502: WYSIWYG: Add CKEditor module to core, we received feedback on code styling, docs, and spelling in the typical Drupal fashion. Here's a patch that addresses all the recommendations there which adds basic docs and makes these code style changes.

CommentFileSizeAuthor
#3 ckeditor.api-d7.patch6.49 KBquicksketch
#2 ckeditor_docs_codestyle-d7.patch8.44 KBquicksketch
#1 ckeditor_docs_codestyle.patch15.38 KBquicksketch

Comments

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new ckeditor_docs_codestyle.patch15.38 KB
quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed
StatusFileSize
new ckeditor_docs_codestyle-d7.patch8.44 KB

Committed to both branches. Here's the backport for D7.

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new ckeditor.api-d7.patch6.49 KB

I missed the api.php file on the patch backport for D7. Here's the one that was committed.

Status: Fixed » Closed (fixed)

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