Skip to content

Feat: new product page #255

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

Open
wants to merge 6 commits into
base: preview
Choose a base branch
from
Open

Feat: new product page #255

wants to merge 6 commits into from

Conversation

asaadam
Copy link
Contributor

@asaadam asaadam commented Jul 13, 2022

https://www.loom.com/share/f7511fb979174f8f929b038e4ecd69b0

  • create new information token card component with compound component
  • provide dash component that can be inserted with text (currently only support left and right text)
  • create new backing information card with table
  • create new calculator component ( minus slider )
  • move my assets to below chart
  • add keyboard shortcut handler for swap ( shift + s )

@asaadam asaadam requested review from pyk and math-marcellino July 13, 2022 07:40
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 13, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2548ce8
Status: ✅  Deploy successful!
Preview URL: https://aea1cc7b.frontend-3mt.pages.dev
Branch Preview URL: https://feat-newproductpage.frontend-3mt.pages.dev

View logs

<p className="text-sm font-semibold leading-4 tracking-[-.02em] text-gray-light-12 dark:text-gray-dark-12">{metadata.debtSymbol}</p>
</td>
<td>
{uiData.status === "loading" && <p className="h-[16px] w-[100px] animate-pulse rounded-[8px] bg-gray-light-3 dark:bg-gray-dark-3"></p>}
Copy link
Contributor

Choose a reason for hiding this comment

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

h-[16px] h-4

rounded-[8px] to rounded-lg

@@ -461,3 +461,15 @@ html {
background-position: 0% 50%;
}
}

kbd {
Copy link
Contributor

Choose a reason for hiding this comment

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

@asaadam what is it for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

for this one ser


const InformationCardExtra = ({ children, className }: { children: ReactNode; className?: string }) => {
return (
<div style={{ marginBottom: "-1rem !important" }} className={`-m-4 rounded-b-[16px] py-2 px-4 text-sm leading-6 text-white dark:text-gray-dark-10 ${className || ""}`}>
Copy link
Contributor

Choose a reason for hiding this comment

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

rounded-b-[16px] to rounded-b-2xl

@pyk
Copy link
Contributor

pyk commented Jul 13, 2022

The navbar is show up when the swap/mint modal open

@math-marcellino
Copy link
Collaborator

math-marcellino commented Jul 13, 2022

The navbar is show up when the swap/mint modal open

Ahh yess sorry I think it's my fault. It's because we change the navbar to be sticky. I'll fix it in the next PR sir... @pyk

@asaadam
Copy link
Contributor Author

asaadam commented Jul 13, 2022

@pyk updated ser

@asaadam asaadam requested a review from pyk July 13, 2022 09:50
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.

None yet

3 participants