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

Add displayed code samples #41

Merged
merged 13 commits into from
Oct 31, 2022
Merged

Add displayed code samples #41

merged 13 commits into from
Oct 31, 2022

Conversation

dvdherron
Copy link
Contributor

@dvdherron dvdherron commented Oct 20, 2022

Show me

image

Provide screenshots/animated gifs/videos if necessary.

@netlify
Copy link

netlify bot commented Oct 20, 2022

Deploy Preview for anchor-polyfill ready!

Name Link
🔨 Latest commit d4c766b
🔍 Latest deploy log https://app.netlify.com/sites/anchor-polyfill/deploys/63601c7d541cae00083d667d
😎 Deploy Preview https://deploy-preview-41--anchor-polyfill.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@dvdherron
Copy link
Contributor Author

@dvdherron dvdherron changed the base branch from main to add-styles October 20, 2022 13:32
@dvdherron dvdherron marked this pull request as ready for review October 20, 2022 15:50
@dvdherron dvdherron changed the base branch from add-styles to main October 20, 2022 16:01
@dvdherron dvdherron changed the base branch from main to add-styles October 20, 2022 16:09
Base automatically changed from add-styles to main October 24, 2022 21:14
@dvdherron
Copy link
Contributor Author

dvdherron commented Oct 28, 2022

@jgerigmeyer Curious to how you see this coming along so far. I've added plain language notes for each (working) example along with relevant code. I hid the notes and code for the two examples that I think aren't working yet?

Seems like the Open Props package is being blocked all of a sudden. I'll see if there's another way to load those styles.
Looks like unpkg is down. That's why the Open Props styles aren't working at the moment.
Fixed

I think the intro paragraph could use another sentence or two explaining the demo? @sanajaved7 and @oluoluoxenfree if you have any feedback on the introduction/example code/and explanation notes that would be great too.

Copy link
Member

@jgerigmeyer jgerigmeyer left a comment

Choose a reason for hiding this comment

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

@dvdherron Looks good to me! I made a few minor wording edits, made one change to the positioning of the scroll example (so that the elements are visible initially without scrolling), and commented out the non-supported examples entirely (so that users aren't confused by them).

It'd be awesome if we could automatically pull the sample CSS from the actual CSS files, so we don't have to remember to keep the demos up-to-date if we change the CSS... But probably not a top priority for now. 👍

@dvdherron
Copy link
Contributor Author

@dvdherron Looks good to me! I made a few minor wording edits, made one change to the positioning of the scroll example (so that the elements are visible initially without scrolling), and commented out the non-supported examples entirely (so that users aren't confused by them).

It'd be awesome if we could automatically pull the sample CSS from the actual CSS files, so we don't have to remember to keep the demos up-to-date if we change the CSS... But probably not a top priority for now. 👍

@jgerigmeyer Your changes look good and make sense. Good call on hiding the non-supported examples completely for now.

And I agree it would be ideal to have the CSS update automatically. Something we can look into later. 👍🏽

* main:
  chore(deps): Automated dependency upgrades
  Update GH Actions output syntax
@jgerigmeyer
Copy link
Member

@sanajaved7 I prepped this for a v0.0.1 release, so once you review it I'll merge and do a new release.

index.html Show resolved Hide resolved
Copy link
Contributor

@sanajaved7 sanajaved7 left a comment

Choose a reason for hiding this comment

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

Deploy preview looks good! 🚀

@jgerigmeyer jgerigmeyer merged commit b1c4d89 into main Oct 31, 2022
@jgerigmeyer jgerigmeyer deleted the code-samples branch October 31, 2022 19:16
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.

3 participants