Skip to content

Conversation

@aatuvai
Copy link
Contributor

@aatuvai aatuvai commented Aug 28, 2025

This PR adds the GuidedSteps component, which can be used to view code along descriptive text on the side.

Five complimenting components are added with the GuidedSteps component:

  • Step: Used to place instruction steps to the GuidedSteps component.
  • StepSection: Starts a new section of Step instructions.
  • File: Groups the code of a particular file / type to its own tab.
  • CodeBlock: marks the lines of code which should be highlighted when the associated instruction Step is active.
  • Checkpoint: used to collect feedback from users while scrolling the code instruction guide (the component is added in Checkpoint component #477)

Rules for GuidedSteps:

  • The main GuidedSteps component can contain as direct children:
  1. Any .md content blocks
  2. Step, StepSection, and File blocks
  3. The File block can only contain CodeBlock blocks

Additionally:

  • Each Step needs to have an id that matches a CodeBlock (with the stepId property).

@github-actions
Copy link

github-actions bot commented Aug 28, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
aatuvai/guided-steps 25434aa 24 ❌FAILED aatuvai-guided-steps 2025-12-02 10:39:54

@aatuvai
Copy link
Contributor Author

aatuvai commented Sep 30, 2025

Preview on this page

@aatuvai aatuvai marked this pull request as ready for review September 30, 2025 09:53
@aatuvai aatuvai force-pushed the aatuvai/guided-steps branch from e37c451 to cce0b9f Compare November 24, 2025 11:02
Comment on lines +9 to +10
"branch": "aatuvai/guided-steps",
"repo_path": "aatuvai/teleport",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Revert before merge

@ptgott
Copy link
Contributor

ptgott commented Dec 1, 2025

Is there a way to split this one up?

For example, it seems like we could review and merge the Checkpoint component separately from the side-by-side layout component.

@aatuvai
Copy link
Contributor Author

aatuvai commented Dec 2, 2025

Is there a way to split this one up?

For example, it seems like we could review and merge the Checkpoint component separately from the side-by-side layout component.

@ptgott Yeah I think you are right, it makes sense to review the Checkpoint in a separate PR. Thanks

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