-
Notifications
You must be signed in to change notification settings - Fork 4
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
base: main
Are you sure you want to change the base?
feat: new javascript example v2 #7
Conversation
There was a problem hiding this 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.
remove single quotes, it is not a string Co-authored-by: Matthew Elwell <[email protected]>
Hey, @matthewelwell! Do i need to modify or add anything else here? |
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! |
New example more friendly to newbies.
@gnumoreno