Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Gathered fixes from abandoned PRs. #6762

Merged
merged 4 commits into from
Mar 2, 2020
Merged

Gathered fixes from abandoned PRs. #6762

merged 4 commits into from
Mar 2, 2020

Conversation

dobooth
Copy link
Contributor

@dobooth dobooth commented Mar 2, 2020

@dobooth dobooth added 2.3.x Magento 2.3 related changes Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies labels Mar 2, 2020
@dobooth dobooth requested a review from keharper March 2, 2020 16:15
@@ -135,7 +135,7 @@ require([

## Calling JS components requiring initialization in JS files {#js_widget_init}

To call a widget in JS code, use a notation similar to the following ([accordion]({{ page.baseurl }}/javascript-dev-guide/widgets/widget_accordion.html) widget is intiialized on the `[data-role=example]` element as illustration):
To call a widget in JS code, use a notation similar to the following ([accordion]({{ page.baseurl }}/javascript-dev-guide/widgets/widget_accordion.html) widget is initialized on the `[data-role=example]` element as illustration):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't even read correctly. Needs work.

dobooth and others added 2 commits March 2, 2020 11:19
Co-Authored-By: hguthrie <hguthrie@users.noreply.github.com>
…s-enable.md

Co-Authored-By: hguthrie <hguthrie@users.noreply.github.com>
@@ -135,7 +135,7 @@ require([

## Calling JS components requiring initialization in JS files {#js_widget_init}

To call a widget in JS code, use a notation similar to the following ([accordion]({{ page.baseurl }}/javascript-dev-guide/widgets/widget_accordion.html) widget is intiialized on the `[data-role=example]` element as illustration):
To call a widget in JS code, use a notation similar to the following ([accordion]({{ page.baseurl }}/javascript-dev-guide/widgets/widget_accordion.html) widget is initialized on the `[data-role=example]` element as illustration):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
To call a widget in JS code, use a notation similar to the following ([accordion]({{ page.baseurl }}/javascript-dev-guide/widgets/widget_accordion.html) widget is initialized on the `[data-role=example]` element as illustration):
To call a widget in JS code, use a notation similar to the following ([accordion]({{ page.baseurl }}/javascript-dev-guide/widgets/widget_accordion.html) widget that is initialized on the `[data-role=example]` element below:

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@dobooth
Copy link
Contributor Author

dobooth commented Mar 2, 2020

running tests

@dobooth dobooth merged commit 6dfbfe2 into master Mar 2, 2020
@ghost
Copy link

ghost commented Mar 2, 2020

Hi @dobooth, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@dobooth dobooth deleted the db_fixes branch March 2, 2020 18:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.3.x Magento 2.3 related changes Editorial Typo and grammar fixes or minor rewrites to correct inaccuracies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants