Skip to content

Conversation

@aditya520
Copy link
Member

Summary

Rationale

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@vercel
Copy link

vercel bot commented Nov 5, 2025

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

Project Deployment Preview Comments Updated (UTC)
component-library Ready Ready Preview Comment Nov 6, 2025 6:37pm
developer-hub Ready Ready Preview Comment Nov 6, 2025 6:37pm
5 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
api-reference Skipped Skipped Nov 6, 2025 6:37pm
entropy-explorer Skipped Skipped Nov 6, 2025 6:37pm
insights Skipped Skipped Nov 6, 2025 6:37pm
proposals Skipped Skipped Nov 6, 2025 6:37pm
staking Skipped Skipped Nov 6, 2025 6:37pm

@aditya520 aditya520 changed the title chore(dev-hub) Integration Card First draft chore(dev-hub) Component: Integration Card Nov 5, 2025
z-index: 0;
}

.barChart {
Copy link
Contributor

Choose a reason for hiding this comment

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

is this used somewhere?

height: 7rem; // 88px - fixed height
}

.decorativeIcon {
Copy link
Contributor

Choose a reason for hiding this comment

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

is this used somewhere?

Comment on lines +60 to +61
{ label: "Browse Supported Feeds", href: "/price-feeds" },
{ label: "Error Codes", href: "/price-feeds" },
Copy link
Contributor

Choose a reason for hiding this comment

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

fyi these are not the right links I'm assuming

Comment on lines +126 to +132
.featureShadow {
position: absolute;
inset: 0;
pointer-events: none;
box-shadow: inset 0 0 4px 0 rgb(255 255 255 / 25%);
border-radius: 1.32rem; // 21px
}
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we need this?

font-size: 1rem; // 16px
font-weight: 400;
line-height: 1.65;
letter-spacing: -0.01em; // -0.16px
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think any of these letter spacing rules are relevant tbh

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.

3 participants