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

[Feat] Add dashboard page #106

Merged
merged 9 commits into from
Sep 30, 2024
Merged

Conversation

sanieni6
Copy link
Contributor

Pull Request

Changes description

  1. Create the dashboard page
  2. Create a hook to get the height and width of any component
  3. refactor the animation to make it works with the fund card and fix the x-scroling in the root page

Note: the dashboard page is fully responsive

Current output

  1. Dashboard page
    image

Time spent breakdown

1 hour

Comments

Any other details you might want to share.

>
<div className="flex flex-col items-start justify-between gap-4 md:gap-6">
<p className=" text-sm md:text-base lg:text-lg text-white font-light leading-[22px] md:leading-[25px] lg:leading-[27.6px]">
{fund.type} {fund.type === "Project" ? "🚀" : "🎗️"}
Copy link
Member

@adrianvrj adrianvrj Sep 29, 2024

Choose a reason for hiding this comment

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

Could you please use the HTML representation of the emojis:
rocket: U+1F680
heart: U+1FAC0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Solved!

@sanieni6 sanieni6 requested a review from adrianvrj September 30, 2024 13:15
<div className="container mx-auto p-6 min-h-screen">
<h1 className="text-4xl font-bold mb-6 flex items-center">
Latest Funds
<span className="ml-2 text-yellow-400">✨</span>
Copy link
Member

Choose a reason for hiding this comment

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

Please use the HTML code for this emoji here too 🙏🏻

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@sanieni6 sanieni6 requested a review from adrianvrj September 30, 2024 13:58
@adrianvrj
Copy link
Member

LGTM! tysm sir

@adrianvrj adrianvrj merged commit 87f75cf into web3wagers:dev Sep 30, 2024
4 checks passed
@sanieni6
Copy link
Contributor Author

My first ever OSS man, thank you so much guys, i am gonna portrait this PR

@adrianvrj
Copy link
Member

My first ever OSS man, thank you so much guys, i am gonna portrait this PR

first of many, congrats homie! 🥳

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.

[feat] Add Dashboard page
2 participants