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

Cindy stats page #65

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

Cindy stats page #65

wants to merge 6 commits into from

Conversation

cindyzli
Copy link
Collaborator

Summary

Finished working on task A:

design edits

Notes

Copy link

vercel bot commented Feb 1, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
perfect-match-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 1, 2025 2:22am

@huajie-zhong
Copy link
Collaborator

I was thinking if the charts can have a border, similar to how it looked like on figma. This should be doable using tailwind
image

d="M0,60 C16.67,40 33.33,40 50,60 C66.67,80 83.33,80 100,60 C116.67,40 133.33,40 150,60 C166.67,80 183.33,80 200,60 C216.67,40 233.33,40 250,60 C266.67,80 283.33,80 300,60 C316.67,40 333.33,40 350,60 C366.67,80 383.33,80 400,60 C416.67,40 433.33,40 450,60 C466.67,80 483.33,80 500,60 C516.67,40 533.33,40 550,60 C566.67,80 583.33,80 600,60 C616.67,40 633.33,40 650,60 C666.67,80 683.33,80 700,60 C716.67,40 733.33,40 750,60 C766.67,80 783.33,80 800,60 C816.67,40 833.33,40 850,60 C866.67,80 883.33,80 900,60 C916.67,40 933.33,40 950,60 C966.67,80 983.33,80 1000,60 C1016.67,40 1033.33,40 1050,60 C1066.67,80 1083.33,80 1100,60 C1116.67,40 1133.33,40 1150,60 C1166.67,80 1183.33,80 1200,60 V120 H0 Z"
fill="#f7a4af"
></path>
</svg>
</div>
</div>

{year === 2022 ? <Stats2022 /> : <Stats2023 />}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I just realized that the FAQ and Thanks for taking Perfect Match this year and don't be afraid to shoot your shot! are inside each Stats2022 and Stats2023. For uniformity and easier changes in the future, would you mind bring them out to the stat page?

@huajie-zhong
Copy link
Collaborator

Also, I realized that there is a lot of manually setting horizontal margin to center items. This can be achieved better by setting width to be full and using justify-content. Setting manually margin is painful and does not align well in the middle.

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