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

[Sistent] Select component doc #6177

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

nganphan123
Copy link

@nganphan123 nganphan123 commented Jan 6, 2025

Description

This PR fixes #5957

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@nganphan123 nganphan123 force-pushed the select-component-doc branch 2 times, most recently from 522ad3b to 499335b Compare January 11, 2025 10:39
@nganphan123 nganphan123 marked this pull request as ready for review January 11, 2025 10:39
@l5io
Copy link
Contributor

l5io commented Jan 11, 2025

🚀 Preview for commit 499335b at: https://67824d17b7b47945fd85e5f8--layer5.netlify.app

@nganphan123
Copy link
Author

Hi @Vidit-Kushwaha , I finished adding the Select component. Could you help review the PR? Thank you!

@vishalvivekm
Copy link
Contributor

@nganphan123
Thank you for your contribution!
Let's discuss this during the website call on Monday at 7 AM CT.

adding it as an agenda item to the meeting minutes.

@vishalvivekm vishalvivekm changed the title Select component doc [Sistent] Select component doc Jan 13, 2025
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
Signed-off-by: nganphan123 <[email protected]>
@l5io
Copy link
Contributor

l5io commented Jan 21, 2025

🚀 Preview for commit 4e9faf2 at: https://678f00671bfff88dee48846d--layer5.netlify.app

Copy link
Contributor

@Vidit-Kushwaha Vidit-Kushwaha left a comment

Choose a reason for hiding this comment

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

Else look good to me

Copy link
Contributor

Choose a reason for hiding this comment

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

We can make this components generic so we can use them through out different component documentation

@@ -0,0 +1,13 @@
import React from "react";
Copy link
Member

Choose a reason for hiding this comment

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

no need of separate file for header. There is file where all components are mentioned.

import React, { useState } from "react";
import Code from "../../../../../components/CodeBlock";

export const CodeBlock = ({ name, code }) => {
Copy link
Member

Choose a reason for hiding this comment

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

no need of this. using existing code block component

import TabButton from "../../../../../reusecore/Button";
import React from "react";
import { useLocation } from "@reach/router";
export default function SectionNav() {
Copy link
Member

Choose a reason for hiding this comment

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

what is use of this file? Routing is done in one of same file where you have defined component. Checkout existing components.

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

Successfully merging this pull request may close these issues.

[Sistent] Add Select component to the sistent components page
5 participants