Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use input event listener for word counter example closes #51 #80

Merged
merged 2 commits into from
Oct 29, 2024

Conversation

MiB-1
Copy link
Contributor

@MiB-1 MiB-1 commented Oct 19, 2024

Word counter example
As mentioned in the issue instead of using setInterval we can use input event listener.

Closes #51

@bsmth
Copy link
Member

bsmth commented Oct 28, 2024

Thanks a lot for opening this one. BTW, we have the following in content that also needs to be updated: https://github.com/mdn/content/blob/ca1fc680cd7c7c04bed18d8ab1fa3988bf7e972e/files/en-us/web/api/element/attachshadow/index.md?plain=1#L163-L166

@bsmth bsmth added the needs content update Needs update to the content to support this change. label Oct 28, 2024
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Nice, works well locally after checking out the branch 👍🏻

MiB-1 added a commit to MiB-1/content that referenced this pull request Oct 28, 2024
Updating the content according to the example
mdn/web-components-examples#80
@MiB-1
Copy link
Contributor Author

MiB-1 commented Oct 28, 2024

thanks @bsmth , here is the link for updating the content repo:

@MiB-1 MiB-1 requested a review from Josh-Cena October 28, 2024 15:52
bsmth pushed a commit to mdn/content that referenced this pull request Oct 29, 2024
* Update content according to the example

Updating the content according to the example
mdn/web-components-examples#80

* fix lint

* Update files/en-us/web/api/element/attachshadow/index.md

---------

Co-authored-by: Joshua Chen <[email protected]>
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

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

Super, thanks a lot!

@MiB-1
Copy link
Contributor Author

MiB-1 commented Oct 29, 2024

Super, thanks a lot!

thanks a lot, looking forward to contribute more. may I know when this be PR will get merge?

@bsmth
Copy link
Member

bsmth commented Oct 29, 2024

Merging now!

@bsmth bsmth merged commit 2e9fbac into mdn:main Oct 29, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs content update Needs update to the content to support this change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Falsy comment in the word-count-web-component javascript file.
4 participants