Commit 68baffe7 authored by alexpott's avatar alexpott

Issue #2385111 by tadityar, Wim Leers: Bartik's CKEditor iframe stylesheet is...

Issue #2385111 by tadityar, Wim Leers: Bartik's CKEditor iframe stylesheet is not being loaded, hence image captions look broken
parent 6c466692
......@@ -91,7 +91,7 @@ function _ckeditor_theme_css($theme = NULL) {
}
}
if (isset($info['base theme'])) {
$css = array_merge(_ckeditor_theme_css($info['base theme'], $css));
$css = array_merge(_ckeditor_theme_css($info['base theme']), $css);
}
}
return $css;
......
......@@ -262,7 +262,11 @@ function testBuildContentsCssJSSetting() {
$expected[] = file_create_url('core/modules/ckeditor/tests/modules/ckeditor_test.css');
$this->assertIdentical($expected, $this->ckeditor->buildContentsCssJSSetting($editor), '"contentsCss" configuration part of JS settings built correctly while a hook_ckeditor_css_alter() implementation exists.');
// @todo test coverage for _ckeditor_theme_css(), by including a custom theme in this test with a "ckeditor_stylesheets" entry in its .info file.
// Enable the Bartik theme, which specifies a CKEditor stylesheet.
\Drupal::service('theme_handler')->install(['bartik']);
\Drupal::config('system.theme')->set('default', 'bartik');
$expected[] = file_create_url('core/themes/bartik/css/ckeditor-iframe.css');
$this->assertIdentical($expected, $this->ckeditor->buildContentsCssJSSetting($editor), '"contentsCss" configuration part of JS settings built correctly while a theme providing a CKEditor stylesheet exists.');
}
/**
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment