Skip to content

HOV-50 | Added New component Breadcrumb #133

Merged
pruthvi2103 merged 16 commits intoantstackio:mainfrom
AnilHash:main
Oct 13, 2022
Merged

HOV-50 | Added New component Breadcrumb #133
pruthvi2103 merged 16 commits intoantstackio:mainfrom
AnilHash:main

Conversation

@AnilHash
Copy link
Copy Markdown
Contributor

Initial commit for Breadcrumb Component.

Description
Breadcrumb component.

Motivation and Context
Adds new component to arsenal of hover.

How Has This Been Tested?
Has been tested manually.

Screenshots (if appropriate):
Types of changes
[x] New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

@netlify
Copy link
Copy Markdown

netlify bot commented Oct 13, 2022

Deploy Preview for hover-design ready!

Name Link
🔨 Latest commit 918a6be
🔍 Latest deploy log https://app.netlify.com/sites/hover-design/deploys/6347e2e146396b0008229d05
😎 Deploy Preview https://deploy-preview-133--hover-design.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Copy Markdown
Contributor

@pruthvi2103 pruthvi2103 left a comment

Choose a reason for hiding this comment

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

One small change

Comment on lines +8 to +17
type crumb = {
title: string;
href?: string;
};

export interface BreadcrumbProps {
crumbs: Array<crumb>;
separator?: string | React.ReactNode;
className?: string;
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

please move this into a seperate file called breadcrumb.types.ts

@pruthvi2103 pruthvi2103 changed the title Added New component Breadcrumb HOV-50 | Added New component Breadcrumb Oct 13, 2022
@pruthvi2103 pruthvi2103 merged commit bd4c521 into antstackio:main Oct 13, 2022
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.

2 participants