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

feat: new javascript example v2 #7

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

MatheusLasserre
Copy link

New example more friendly to newbies.

@gnumoreno

Copy link

@matthewelwell matthewelwell left a comment

Choose a reason for hiding this comment

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

This looks good on the whole, I like how readable the code is in index.js. I've added a few minor comments.

javascript-vanilla-v2/main.js Show resolved Hide resolved
javascript-vanilla-v2/main.js Outdated Show resolved Hide resolved
javascript-vanilla-v2/main.js Outdated Show resolved Hide resolved
javascript-vanilla-v2/main.js Outdated Show resolved Hide resolved
gnumoreno and others added 2 commits January 10, 2024 16:59
remove single quotes, it is not a string

Co-authored-by: Matthew Elwell <[email protected]>
@MatheusLasserre
Copy link
Author

Hey, @matthewelwell! Do i need to modify or add anything else here?

@kyle-ssg
Copy link
Member

kyle-ssg commented Apr 5, 2024

I'd really love to standardise all of the JS examples with shared markup, CSS, content and demonstation. Perhaps what I'll do is use some of this PR as inspiration. Thanks very much for this

@MatheusLasserre
Copy link
Author

I'd really love to standardise all of the JS examples with shared markup, CSS, content and demonstation. Perhaps what I'll do is use some of this PR as inspiration. Thanks very much for this

If this one was approved, I was thinking about gradually doing for the others JS Examples in my spare time. If you wanted, of course. But anyway, glad it helped!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants