Staging to Main: Homepage REVAMP #289
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
removed the old code only for this section and wrote a new one with tailwind css
removed fixed keyword to allow the background image to scroll with the page.
added the learn web3 and partner with banners.
the banner section is complete
added the new dd logo resized all text size and the social icons
changed the design of the navigation bar also added the posts as a new nav option.
added the gradient border to the banners
updated partner count, adjusted post symbol and full stop graphic in hero, left aligned footer text.
…d clickability across all pages
In the landing page code, added the background image in the div. This caused it to be removed from other pages.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…into feat/staging-update
Feat/staging update from main
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://staging-academyv2.vercel.app/ |
|
Just on a call with @rohann06 and in essence the PR is good to merge. However a new PR is needed because the images are taking too long to load since they are quite big/heavy. |
elPiablo
approved these changes
Dec 17, 2024
There was a problem hiding this comment.
On a call with @rohann06 here and the images (too big/heavy) are loading to slowly, so a fresh PR is needed to solve that. But we can go ahead and merge this PR now.
cc @Markkos89
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes