-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: tab new component #6280
base: master
Are you sure you want to change the base?
feat: tab new component #6280
Conversation
🚀 Preview for commit e885203 at: https://67c6c4ce7fbc5fe040bc2a5d--layer5.netlify.app |
Signed-off-by: Jaishree2310 <jaishrees23102001@gmail.com>
e885203
to
05adcc9
Compare
🚀 Preview for commit 05adcc9 at: https://67c70ff436fb06006e5c0da5--layer5.netlify.app |
? "active" | ||
: "" | ||
} | ||
onClick={() => navigate("/projects/sistent/components/tab")} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Jaishree2310 There is a typo here, it redirects to an error page.
It should be: /projects/sistent/components/tabs
onClick={() => navigate("/projects/sistent/components/tab")} | |
onClick={() => navigate("/projects/sistent/components/tabs")} |
@Jaishree2310, I might have missed some updates here, but I’d recommend checking your implementation’s UI by running the website locally. It’s important to take the time to understand the codebase before working on issues. I also noticed a similar concern with the avatar component, especially in the code section. It would be great if you could review your work carefully before submitting. This will help reduce the number of improvement suggestions needed. I truly appreciate your efforts, and I’m sure you’ll get better with time! You will find your commits preview, similar to this one: ![]() |
Description
This PR fixes #5961
Notes for Reviewers
Signed commits