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

New landing page #537

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

New landing page #537

wants to merge 1 commit into from

Conversation

Tbaut
Copy link
Collaborator

@Tbaut Tbaut commented May 15, 2024

No description provided.

@Tbaut
Copy link
Collaborator Author

Tbaut commented May 15, 2024

@sweetpea22 I've isolated the page and components about the landing page in a dedicated dir. It looks eventually messy in the app structure, but I can change it later, my goal is to provide and environment that is easy for you to jump in.

so basically:

  • you can build everything inside this LandingComponents
  • to see the result in the app either:
    • you have no connected or watched account, then this should be the landing page you see right away.
    • you have a connected account already with multisigs, then go to "About" in the top menu. This will be the live behavior, so that if you use Multix already, we want to show what you care about right away. If you don't, then we tell you more about Multix while inviting you to connect an account.
  • I've built a super simple Hero component, with I think a good amount of things that are interesting such as:
    • the buttons to connect etc, so you don't have to figure it out
    • a basic design so that you see how it's done throughout the app, basically style component with pure css
    • some usage of some common variables.
  • forget about the tests, I'll fix them after for you.

Unless you have it already, I highly recommend you to use the vscode-styled-components plugin to have autocompletion and code styling.

You should see that:

image

@sweetpea22
Copy link

perfect, thank you!

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