Skip to content

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.

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