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

Optimization #63

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

Optimization #63

wants to merge 9 commits into from

Conversation

jonbingkim
Copy link
Contributor

Summary

Changes made-
Next/font utilized for faster load time. No more layout shifts based on load and FOUT (flash of unstyled text)
Took out loading animation, currently shows return

loading
to minimize any more loading time
guild logo added to footer linking to potlock guild
centered login button with current header on mobile
fixed some margins with text on feeds
added routes to tabs on feeds, go from feeds => activity emoji => takes you to tab?=...

Related Issue

#56
#57
#44

Type of Change

  • Refactor
  • Bug fix
  • New feature

Testing Steps

npm run dev
do a lighthouse test after building to check performance metrics
routes to check =>
home page
feeds

Copy link

vercel bot commented Nov 13, 2024

@jonbingkim is attempting to deploy a commit to the PotLock Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant