Instructions for adding CKEditor stylesheets are outdated

Good catch, and thanks for filing the issue! Seems like a good Novice project to update the docs here.

Unassigning because there has been no progress on this issue in more than two weeks, and it looks like a good candidate for the Drupal North 2016 coding sprint tomorrow.

@mayurjadhav, if you are still working on this, please assign it to yourself again!

latest changes applied.

Yep the docs should be updated. The patch looks good.

Patch looks perfect, +1 for RTBC

This is a documentation only patch, and so this is a random test failure.

I am triggering a retest

If it touches CKEditor, then please put it in that component, so I can review it as the module maintainer.

+++ b/core/modules/ckeditor/ckeditor.module
@@ -68,13 +68,15 @@ function ckeditor_ckeditor_css_alter(array &$css, Editor $editor) {
- * Retrieves the default theme's CKEditor stylesheets defined in the .info file.
+ * Retrieves the default theme's CKEditor stylesheets defined in the .info.yml
+ * file.

Violates 80 cols rule.

Proposed message:

Retrieves the default theme's CKEditor stylesheets.

fixed.

Thanks!

Committed and pushed 4e9ed76 to 8.2.x and 619cbc3 to 8.1.x. Thanks!

+++ b/core/modules/ckeditor/ckeditor.module
@@ -68,13 +68,14 @@ function ckeditor_ckeditor_css_alter(array &$css, Editor $editor) {
+ * ckeditor_stylesheets:
+ *  - css/ckeditor-iframe.css

The CSS line needs to be indented by one more space. Fixed on commit - see interdiff.

Thanks, @Cottser :)