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

Add Light/Dark Mode and remove unnecessary files #26

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from

Conversation

Megumiiiiii
Copy link

No description provided.

Copy link

vercel bot commented Dec 19, 2023

@Megumiiiiii is attempting to deploy a commit to the Avail Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Dec 19, 2023

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

Name Status Preview Comments Updated (UTC)
light-client-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 8, 2024 1:21pm

@abheektripathy
Copy link
Member

abheektripathy commented Dec 19, 2023

hey @Megumiiiiii thanks for your interest in the project.

few suggestions-

  • fix the matrix colors, plus loading bar colors for light mode as well
Screenshot 2023-12-19 at 9 44 34 PM
  • use the day and night logos rather than the text "switch theme mode" for the button for example-
Screenshot 2023-12-19 at 9 47 50 PM
  • can we have this color[#1E2129] for dark mode rather than black
  • for mobile sizes, the logo dosen't show up, can you fix that?
Screenshot 2023-12-19 at 9 50 48 PM

edit: also, the build fails due to the pnpm-lock.yaml not being up-to-date with the package.json can you run pnpm i before you push?

@Megumiiiiii
Copy link
Author

how about that?

Screenshot_553
Screenshot_554

@Megumiiiiii
Copy link
Author

  • fix the matrix colors
  • use the day and night logos
  • background color[#1E2129]
  • mobile views logo won't show up

pnpm-lock has also been updated

mobile

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