diff --git a/ghost/admin/app/styles/components/koenig.css b/ghost/admin/app/styles/components/koenig.css index dc2f8443b9..48aca90696 100644 --- a/ghost/admin/app/styles/components/koenig.css +++ b/ghost/admin/app/styles/components/koenig.css @@ -138,6 +138,9 @@ stroke-width: 2px; stroke: color(var(--lightgrey) l(-10%)); } +.koenig-toolbar-btn-list svg path { + fill: color(var(--lightgrey) l(-10%)); +} .koenig-toolbar-btn:hover, .koenig-toolbar-btn.selected { @@ -145,10 +148,16 @@ cursor: pointer; text-shadow: #000 0 1px 6px; } +.koenig-toolbar-btn-list.selected svg path { + fill: #fff; +} .koenig-toolbar-btn:hover svg g { stroke: #fff; } +.koenig-toolbar-btn-list:hover svg path { + fill: #fff; +} .koenig-toolbar-btn-bold { font-weight: 700; diff --git a/ghost/admin/lib/koenig-editor/addon/components/koenig-editor.js b/ghost/admin/lib/koenig-editor/addon/components/koenig-editor.js index 30bc408a1d..914c1e73b0 100644 --- a/ghost/admin/lib/koenig-editor/addon/components/koenig-editor.js +++ b/ghost/admin/lib/koenig-editor/addon/components/koenig-editor.js @@ -366,20 +366,6 @@ export default Component.extend({ }); }, - replaceWithListSection(listType, range) { - let editor = this.get('editor'); - let {head: {section}} = range; - - editor.run((postEditor) => { - let {builder} = postEditor; - let item = builder.createListItem(); - let listSection = builder.createListSection(listType, [item]); - - postEditor.replaceSection(section, listSection); - postEditor.setRange(listSection.headPosition()); - }); - }, - selectCard(card) { this.selectCard(card); }, diff --git a/ghost/admin/lib/koenig-editor/addon/components/koenig-plus-menu.js b/ghost/admin/lib/koenig-editor/addon/components/koenig-plus-menu.js index ddcad39663..f0b5cef240 100644 --- a/ghost/admin/lib/koenig-editor/addon/components/koenig-plus-menu.js +++ b/ghost/admin/lib/koenig-editor/addon/components/koenig-plus-menu.js @@ -28,7 +28,6 @@ export default Component.extend({ // closure actions replaceWithCardSection() {}, - replaceWithListSection() {}, style: computed('top', function () { return htmlSafe(`top: ${this.get('top')}px`); @@ -88,14 +87,6 @@ export default Component.extend({ this.replaceWithCardSection(cardName, range); this._hideButton(); - this._hideMenu(); - }, - - replaceWithListSection(listType) { - let range = this._editorRange; - - this.replaceWithListSection(listType, range); - this._hideMenu(); } }, diff --git a/ghost/admin/lib/koenig-editor/addon/components/koenig-slash-menu.js b/ghost/admin/lib/koenig-editor/addon/components/koenig-slash-menu.js index 5329e96c15..75635b2697 100644 --- a/ghost/admin/lib/koenig-editor/addon/components/koenig-slash-menu.js +++ b/ghost/admin/lib/koenig-editor/addon/components/koenig-slash-menu.js @@ -36,20 +36,6 @@ const ITEM_MAP = [ matches: ['divider', 'horizontal-rule', 'hr'], type: 'card', replaceArg: 'hr' - }, - { - label: 'Bullet list', - icon: 'koenig/list-bullets', - matches: ['list-bullet', 'bullet', 'ul'], - type: 'list', - replaceArg: 'ul' - }, - { - label: 'Number list', - icon: 'koenig/list-number', - matches: ['list-number', 'number', 'ol'], - type: 'list', - replaceArg: 'ol' } ]; @@ -74,7 +60,6 @@ export default Component.extend({ // closure actions replaceWithCardSection() {}, - replaceWithListSection() {}, style: computed('top', function () { return htmlSafe(`top: ${this.get('top')}px`); @@ -116,8 +101,6 @@ export default Component.extend({ if (item.type === 'card') { this.replaceWithCardSection(item.replaceArg, range); - } else if (item.type === 'list') { - this.replaceWithListSection(item.replaceArg, range); } this._hideMenu(); diff --git a/ghost/admin/lib/koenig-editor/addon/components/koenig-toolbar.js b/ghost/admin/lib/koenig-editor/addon/components/koenig-toolbar.js index 59cb555c51..c73de8832e 100644 --- a/ghost/admin/lib/koenig-editor/addon/components/koenig-toolbar.js +++ b/ghost/admin/lib/koenig-editor/addon/components/koenig-toolbar.js @@ -104,6 +104,8 @@ export default Component.extend({ } }, + /* private methods ------------------------------------------------------ */ + _toggleVisibility: task(function* () { // debounce for 100ms to account for "click to deselect" otherwise we // run twice and the fade out animation jumps position diff --git a/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-editor.hbs b/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-editor.hbs index f919d3be32..ee6c68fb7d 100644 --- a/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-editor.hbs +++ b/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-editor.hbs @@ -16,7 +16,6 @@ editor=editor editorRange=selectedRange replaceWithCardSection=(action "replaceWithCardSection") - replaceWithListSection=(action "replaceWithListSection") }} {{!-- slash menu popup --}} @@ -24,7 +23,6 @@ editor=editor editorRange=selectedRange replaceWithCardSection=(action "replaceWithCardSection") - replaceWithListSection=(action "replaceWithListSection") }} {{!-- all component cards wormholed into the editor canvas --}} diff --git a/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-plus-menu.hbs b/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-plus-menu.hbs index 4147173d93..be5ec1a815 100644 --- a/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-plus-menu.hbs +++ b/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-plus-menu.hbs @@ -27,13 +27,5 @@
{{inline-svg "koenig/divider"}}
Divider
-
-
{{inline-svg "koenig/list-bullets"}}
-
Bullet list
-
-
-
{{inline-svg "koenig/list-number"}}
-
Number list
-
{{/if}} diff --git a/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-toolbar.hbs b/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-toolbar.hbs index 408756e7c5..67e100075e 100644 --- a/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-toolbar.hbs +++ b/ghost/admin/lib/koenig-editor/addon/templates/components/koenig-toolbar.hbs @@ -15,25 +15,6 @@ > I - - - - - -{{!-- block buttons --}} + + + + + + + + +