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

added documentation of Link Component #6013

Closed

Conversation

NishantSinghhhhh
Copy link
Contributor

Description

Component Name : Link component present in Sistent component folder ,

This PR addresses various enhancements and features, including:

Documentation : I have written Documentation for the Link Component Used in Sistent Repository
Link Variants: Added different styles for links, including underlined and colored versions to enhance navigation and visibility.

Security Practices: Documented best practices for using links with target="_blank", highlighting the importance of rel="noopener" and rel="noreferrer" attributes to improve security.

Notes for Reviewers

Please pay special attention to the documentation and security notes for external links. Feedback on these changes would be greatly appreciated.

preview:
Screenshot 2024-10-20 235214

Signed commits**

Yes, I signed my commits.

@github-actions github-actions bot added the area/projects An issue relating to Layer5 initiatives (projects) label Oct 20, 2024
@vishalvivekm
Copy link
Member

@NishantSinghhhhh Thank you for your contribution!
Let's discuss this during the website call on Monday at 5:30 PM IST (7:00 AM CT).

Please add it as an agenda item to the meeting minutes.

@vishalvivekm vishalvivekm added the pr/hold Do not merge this PR label Oct 21, 2024
@nebula-aac
Copy link

@NishantSinghhhhh Can you check and fix the errors due to the github actions job?

https://github.com/layer5io/layer5/actions/runs/11429709327/job/31796389728

@NishantSinghhhhh
Copy link
Contributor Author

@vishalvivekm Sir I am not able to edit the doc that you put in your previous comment ,thus I am not able to add my name in it

@NishantSinghhhhh
Copy link
Contributor Author

@nebula-aac #6015 , I have created 1 more PR that passes all the test except the kanvas test , can you please review it
Screenshot 2024-10-21 124816

@vishalvivekm
Copy link
Member

vishalvivekm commented Oct 21, 2024

Duplicate of #6015

@vishalvivekm vishalvivekm marked this as a duplicate of #6015 Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/projects An issue relating to Layer5 initiatives (projects) pr/hold Do not merge this PR project/sistent
Development

Successfully merging this pull request may close these issues.

4 participants