Skip to content

Commit 660fd0a

Browse files
committed
MAGETWO-90441: Allow inline editing of Heading from stage
- remove redundancy in named methods
1 parent 8503675 commit 660fd0a

File tree

5 files changed

+16
-16
lines changed

5 files changed

+16
-16
lines changed

app/code/Magento/PageBuilder/view/adminhtml/web/js/content-type/heading/preview.js

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

app/code/Magento/PageBuilder/view/adminhtml/web/js/toolbar.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

app/code/Magento/PageBuilder/view/adminhtml/web/template/content-type/heading/default/preview.html

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,22 +12,22 @@
1212
<render args="template" />
1313
</with>
1414
<if args="data.main.heading_type() === 'h1'">
15-
<h1 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onHeadingRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h1>
15+
<h1 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h1>
1616
</if>
1717
<if args="data.main.heading_type() === 'h2'">
18-
<h2 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onHeadingRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h2>
18+
<h2 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h2>
1919
</if>
2020
<if args="data.main.heading_type() === 'h3'">
21-
<h3 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onHeadingRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h3>
21+
<h3 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h3>
2222
</if>
2323
<if args="data.main.heading_type() === 'h4'">
24-
<h4 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onHeadingRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h4>
24+
<h4 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h4>
2525
</if>
2626
<if args="data.main.heading_type() === 'h5'">
27-
<h5 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onHeadingRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h5>
27+
<h5 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h5>
2828
</if>
2929
<if args="data.main.heading_type() === 'h6'">
30-
<h6 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onHeadingRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h6>
30+
<h6 attr="data.main.attributes" ko-style="data.main.style" css="data.main.css" afterRender="function (element) { onRender.call($data, element); }" data-bind="liveEdit: { field: 'heading_text', placeholder: $t('Edit Heading Text') }"></h6>
3131
</if>
3232
</div>
3333
</div>

app/code/Magento/PageBuilder/view/adminhtml/web/ts/js/content-type/heading/preview.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,16 +32,16 @@ export default class Heading extends BasePreview {
3232
) {
3333
super(parent, config, observableUpdater);
3434

35-
this.onToolbarFocusIn = Toolbar.onToolbarFocusIn;
36-
this.onToolbarFocusOut = Toolbar.onToolbarFocusOut;
35+
this.onToolbarFocusIn = Toolbar.onFocusIn;
36+
this.onToolbarFocusOut = Toolbar.onFocusOut;
3737
}
3838

3939
/**
4040
* On render init the tabs widget
4141
*
4242
* @param {Element} element
4343
*/
44-
public onHeadingRender(element: Element) {
44+
public onRender(element: Element) {
4545
this.element = element;
4646
}
4747

app/code/Magento/PageBuilder/view/adminhtml/web/ts/js/toolbar.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ export default {
1414
* @param {Preview} context
1515
* @param {Event} event
1616
*/
17-
onToolbarFocusIn(context: Preview, event: Event): void {
17+
onFocusIn(context: Preview, event: Event): void {
1818
const currentContentTypeTarget = event.currentTarget.closest(".pagebuilder-content-type");
1919
$(currentContentTypeTarget).addClass("pagebuilder-toolbar-active");
2020
events.trigger("interaction:start");
@@ -26,7 +26,7 @@ export default {
2626
* @param {Preview} context
2727
* @param {Event} event
2828
*/
29-
onToolbarFocusOut(context: Preview, event: Event): void {
29+
onFocusOut(context: Preview, event: Event): void {
3030
const currentContentTypeTarget = event.currentTarget.closest(".pagebuilder-content-type");
3131
$(currentContentTypeTarget).removeClass("pagebuilder-toolbar-active");
3232
events.trigger("interaction:stop");

0 commit comments

Comments
 (0)