Skip to content

Conversation

@shleewhite
Copy link
Contributor

@shleewhite shleewhite commented Nov 4, 2025

📌 Summary

PREVIEW

If merged, this PR would add a short blurb on how to use Helios components in SFCs.

Note: the syntax highlighting is definitely not perfect, this is a limitation of Prism as they don't have a gts/gjs highlighter. This can be addressed more holistically when we decide to add gts snippets to the component docs pages.

📸 Screenshots

Screenshot 2025-11-04 at 4 24 32 PM Screenshot 2025-11-04 at 4 24 37 PM

🔗 External links

Jira ticket: HDS-5583


💬 Please consider using conventional comments when reviewing this PR.

📋 PCI review checklist
  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.
  • If applicable, I've worked with GRC to document the impact of any changes to security controls.
    Examples of changes to controls include access controls, encryption, logging, etc.
  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.
    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

@shleewhite shleewhite requested review from a team as code owners November 4, 2025 16:45
@vercel
Copy link

vercel bot commented Nov 4, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
hds-showcase Ready Ready Preview Nov 4, 2025 9:30pm
hds-website Ready Ready Preview Nov 4, 2025 9:30pm

@hashibot-hds hashibot-hds added the docs-website Content updates to the documentation website label Nov 4, 2025
Copy link
Contributor

@KristinLBradley KristinLBradley left a comment

Choose a reason for hiding this comment

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

Just submitted one minor suggested edit but other than that looks good!


### Single file components

If you are are using single file components (ie., `.gts` or `.gjs` files), the components need to be individually imported into the file for them to render. All components can be imported from the `@hashicorp/design-system-components/components` path. To use a component's signature, you must import it from the definition file.
Copy link
Contributor

@KristinLBradley KristinLBradley Nov 4, 2025

Choose a reason for hiding this comment

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

Suggested change
If you are are using single file components (ie., `.gts` or `.gjs` files), the components need to be individually imported into the file for them to render. All components can be imported from the `@hashicorp/design-system-components/components` path. To use a component's signature, you must import it from the definition file.
If you are are using single file components (i.e., `.gts` or `.gjs` files), the components need to be individually imported into the file for them to render. All components can be imported from the `@hashicorp/design-system-components/components` path. To use a component's signature, you must import it from the definition file.

See: https://www.wikihow.com/Use-i.e.-in-a-Sentence for more info


interface MyComponentSignature {
Args: {};
Element: HdsFormTextInputFieldSignature;
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be like this?

Suggested change
Element: HdsFormTextInputFieldSignature;
Element: HdsFormTextInputFieldSignature['Element'];


const MyComponent: TOC<MyComponentSignature> = <template>
<HdsFormTextInputField ...attributes as |F|>
<F.Label>Name</F.Label>
Copy link
Contributor

Choose a reason for hiding this comment

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

[suggestion] how about we make the Name an argument?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-website Content updates to the documentation website

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants