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

CW2-32-add-statistics-to-about-us-page #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andrew-l3
Copy link
Collaborator

Why the changes are required?

Per spec

Changes

  • UI changes (as shown); proof of concept - more functionalities to be added.
  • Stats buttons are interactive, alters logo progression indicator.
  • Readme revamped.

Screenshots

screen 2
screen 3

Comments

The about component on the homepage will have to be removed and redesigned (this is purely for the about page).

@andrew-l3 andrew-l3 requested a review from derekxu04 July 25, 2024 06:09
@derekxu04
Copy link
Collaborator

This looks really cool, I really like the animation and I think there's a lot we can use.

I still want to keep up with the Github theme though (try to make it look recognisable as a Github copy), idk if you've seen these sorts of Github profiles but I was thinking something like this:
image
or
image

So I think we can definitely use some of the components you've created here, but maybe stick it in the readme to mimic Github

@derekxu04
Copy link
Collaborator

on hold for now, leave bigger redesign until deployment

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