From 6681b15b3f0d6caa2960ce7aab4827d248b990b0 Mon Sep 17 00:00:00 2001 From: Aleksander Nowodzinski Date: Tue, 6 Feb 2018 14:08:19 +0100 Subject: [PATCH 1/3] Moved content styles from ckeditor5-theme-lark (see ckeditor/ckeditor5-theme-lark#119). --- theme/heading.css | 27 ++++++++++++++++++++++----- 1 file changed, 22 insertions(+), 5 deletions(-) diff --git a/theme/heading.css b/theme/heading.css index 66c26cc..0fe8d43 100644 --- a/theme/heading.css +++ b/theme/heading.css @@ -3,8 +3,25 @@ * For licensing, see LICENSE.md. */ -/* - * Note: This file should contain the wireframe styles only. But since there are no such styles, - * it acts as a message to the builder telling that it should look for the corresponding styles - * **in the theme** when compiling the editor. - */ +.ck-heading_heading1 { + font-size: 1.5em; +} + +.ck-heading_heading2 { + font-size: 1.3em; +} + +.ck-heading_heading3 { + font-size: 1.1em; +} + +/* Using absolute units to make sure each element has the same height and padding. +https://github.com/ckeditor/ckeditor5-heading/issues/63 */ +[class*="ck-heading_"] { + line-height: 18px; + padding: 11px; +} + +[class*="ck-heading_heading"] { + font-weight: bold; +} From d0bf0dc6d890028c6553ac85f2e6f3ebf5438535 Mon Sep 17 00:00:00 2001 From: Aleksander Nowodzinski Date: Tue, 6 Feb 2018 14:13:25 +0100 Subject: [PATCH 2/3] Used absolute font sizze values for heading previews. --- theme/heading.css | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/theme/heading.css b/theme/heading.css index 0fe8d43..74c023b 100644 --- a/theme/heading.css +++ b/theme/heading.css @@ -4,15 +4,15 @@ */ .ck-heading_heading1 { - font-size: 1.5em; + font-size: 20px; } .ck-heading_heading2 { - font-size: 1.3em; + font-size: 17px; } .ck-heading_heading3 { - font-size: 1.1em; + font-size: 14px; } /* Using absolute units to make sure each element has the same height and padding. From 5d235f8450c763cddf9194cd7dbf0fec43aa325c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotrek=20Koszuli=C5=84ski?= Date: Tue, 6 Feb 2018 15:21:18 +0100 Subject: [PATCH 3/3] Follow the sizes defined in the content styles. --- docs/_snippets/features/custom-heading-elements.html | 2 +- docs/features/headings.md | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/_snippets/features/custom-heading-elements.html b/docs/_snippets/features/custom-heading-elements.html index 5e6fead..e63a48f 100644 --- a/docs/_snippets/features/custom-heading-elements.html +++ b/docs/_snippets/features/custom-heading-elements.html @@ -1,7 +1,7 @@ diff --git a/docs/features/headings.md b/docs/features/headings.md index bc9c919..f150d19 100644 --- a/docs/features/headings.md +++ b/docs/features/headings.md @@ -61,7 +61,7 @@ For example, the following editor will support the following two heading options // Styles for the heading in the content and for the dropdown item. h2.fancy, .ck-heading_heading2_fancy { color: #ff0050; - font-size: 1.3em; + font-size: 17px; }