diff --git a/core/modules/ckeditor5/js/ckeditor5.js b/core/modules/ckeditor5/js/ckeditor5.js index 1bd4003cc8..441f6d5f15 100644 --- a/core/modules/ckeditor5/js/ckeditor5.js +++ b/core/modules/ckeditor5/js/ckeditor5.js @@ -371,15 +371,18 @@ ClassicEditor.create(element, editorConfig) .then((editor) => { /** - * Injects a temporary
into CKEditor and then calculates the entire - * height of the amount of the
tags from the passed in rows value. + * Injects a temporary
into CKEditor and then calculates the + * entire height of the amount of the
tags from the passed in rows + * value. * * This takes into account collapsing margins, and line-height of the * current theme. * - * @param {number} - the number of rows. + * @param {number} rows + * The number of rows. * - * @returns {number} - the height of a div in pixels. + * @returns {number} + * The height of a div in pixels. */ function calculateLineHeight(rows) { const element = document.createElement('p'); diff --git a/core/modules/ckeditor5/tests/src/Nightwatch/Tests/ckEditor5EditorHeightTest.js b/core/modules/ckeditor5/tests/src/Nightwatch/Tests/ckEditor5EditorHeightTest.js index cdb24346e1..85976ddcfe 100644 --- a/core/modules/ckeditor5/tests/src/Nightwatch/Tests/ckEditor5EditorHeightTest.js +++ b/core/modules/ckeditor5/tests/src/Nightwatch/Tests/ckEditor5EditorHeightTest.js @@ -6,7 +6,7 @@ module.exports = { after(browser) { browser.drupalUninstall(); }, - 'Ensure CKEditor respects field widget row value': (browser) => { + 'Ensure CKEditor 5 respects field widget row value': (browser) => { browser.drupalLoginAsAdmin(() => { browser // Enable required modules. @@ -63,9 +63,10 @@ module.exports = { ).clientHeight; // We expect height to be 320, but test to ensure that it's greater - // than 300. We want to ensure that we don't hard code a very specific - // value because tests might break if styles change (line-height, etc). - // Note that the default height for CKEditor5 is 47px. + // than 300. We want to ensure that we don't hard code a very + // specific value because tests might break if styles change + // (line-height, etc). Note that the default height for CKEditor 5 + // is 47px. return height > 300; }, [], @@ -137,9 +138,10 @@ module.exports = { ).clientHeight; // We expect height to be 640, but test to ensure that it's greater - // than 600. We want to ensure that we don't hard code a very specific - // value because tests might break if styles change (line-height, etc). - // Note that the default height for CKEditor5 is 47px. + // than 600. We want to ensure that we don't hard code a very + // specific value because tests might break if styles change + // (line-height, etc). Note that the default height for CKEditor 5 + // is 47px. return height > 600; }, [], @@ -157,9 +159,10 @@ module.exports = { ).clientHeight; // We expect height to be 640, but test to ensure that it's greater - // than 600. We want to ensure that we don't hard code a very specific - // value because tests might break if styles change (line-height, etc). - // Note that the default height for CKEditor5 is 47px. + // than 600. We want to ensure that we don't hard code a very + // specific value because tests might break if styles change + // (line-height, etc). Note that the default height for CKEditor 5 + // is 47px. return height > 600; }, [],