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

Initial homepage #59

Merged
merged 6 commits into from
Mar 7, 2023
Merged

Initial homepage #59

merged 6 commits into from
Mar 7, 2023

Conversation

calvellido
Copy link
Member

This PR is adding an initial version of the homepage to be iterating on.

As the site has been evolving through this implementation, it's still lacking a lot of polish, common bits of logic refactoring, responsive versions for most of the components. Also, some more things need to be decided on, like the behaviour and interactions of some components, specifically the cards versions.

This is being submitted to merge to have a foundation to continue working on the rest of the components and sections of the site.

image

@nomisRev
Copy link
Member

nomisRev commented Mar 6, 2023

@calvellido Is this something @israelpzglez should review, or should I "review" how it looks locally?

Copy link

@israelperezglez israelperezglez left a comment

Choose a reason for hiding this comment

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

Good Job! ;)

Copy link
Member

@nomisRev nomisRev left a comment

Choose a reason for hiding this comment

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

Ran the website locally, and looks amazing 😍

@calvellido calvellido merged commit 8897df4 into main Mar 7, 2023
@calvellido calvellido deleted the calvellido-home-page branch March 7, 2023 09:01
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