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

docs(skeleton): reworked examples #2252

Merged
merged 5 commits into from
Jan 19, 2024
Merged

docs(skeleton): reworked examples #2252

merged 5 commits into from
Jan 19, 2024

Conversation

ianmcburnie
Copy link
Contributor

@ianmcburnie ianmcburnie commented Jan 17, 2024

  • This PR contains CSS changes
  • This PR does not contain CSS changes

PLEASE SQUASH

Description

These are documentation changes only for the skeleton examples. Best way to review this is to pull it down locally, run it and read through it and view each example.

Please focus on narrative and nature of examples. Look for factual errors or anything that might be confusing/unclear.

Please do not focus on the example CSS and JS. It's quick and dirty and we can clean that up if we feel the need.

I decided to remove the frames as I didn't feel like they added anything. I do feel that we might want to go back and reformat and make things a little prettier at some point.

Copy link
Contributor

@ArtBlue ArtBlue left a comment

Choose a reason for hiding this comment

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

Looks good overall. Just a couple of minor text suggestions...

docs/_includes/skeleton.html Outdated Show resolved Hide resolved
docs/_includes/skeleton.html Outdated Show resolved Hide resolved
saiponnada
saiponnada previously approved these changes Jan 18, 2024
Copy link
Contributor

@saiponnada saiponnada left a comment

Choose a reason for hiding this comment

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

Concise, engaging & entertaining! Loved it.

docs/skeleton-examples/buffered/example-1/script.js Outdated Show resolved Hide resolved
agliga
agliga previously approved these changes Jan 18, 2024
docs/skeleton-examples/buffered/example-1/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/in-order/example-1a/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/in-order/example-1b/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/in-order/example-2a/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/in-order/example-2b/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/out-of-order/example-1/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/out-of-order/example-2/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/out-of-order/example-3/script.js Outdated Show resolved Hide resolved
docs/skeleton-examples/out-of-order/example-4/script.js Outdated Show resolved Hide resolved
@ianmcburnie ianmcburnie dismissed stale reviews from agliga and saiponnada via cba9721 January 19, 2024 03:29
Co-authored-by: Arthur Khachatryan <[email protected]>
Co-authored-by: Goutham Ponnada <[email protected]>
@ianmcburnie ianmcburnie merged commit 46f8616 into master Jan 19, 2024
2 checks passed
@agliga agliga deleted the docs-skeleton-2 branch April 30, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants