UI Enhancement: Revamp of footer#1772
Conversation
|
@steam-bell-92 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
|
✅ Synchronized metadata from Issue #1684:
|
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the footer hero layout and visual styling, introducing new CTA links and shifting the footer to a light (white) presentation, along with minor blog markdown cleanup.
Changes:
- Restructured footer hero into left/right sections and added Blog + GitHub CTA buttons.
- Overhauled
enhanced-footer.cssto use a white background and dark text, removing gradients/animations in multiple places. - Simplified blog post image embeds by removing empty links around images.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/theme/Footer/Layout/index.tsx | Adds GitHub CTA and reorganizes footer hero markup into left/right sections. |
| src/theme/Footer/Layout/enhanced-footer.css | Reworks footer styling to a white theme and adds CTA/button styles + layout grid. |
| blog/google-icon-update/index.md | Replaces linked images with plain images inside BrowserWindow blocks. |
Comments suppressed due to low confidence (2)
src/theme/Footer/Layout/enhanced-footer.css:1
.footer-iconno longer setsdisplay: flex, but still usesalign-items/justify-content, which will have no effect without a flex/grid layout. Also,coloris duplicated. Reintroducedisplay: flex(or grid) and remove the duplicate property to ensure icons remain centered.
/* Enhanced Footer Styles - COMPLETELY ISOLATED FROM THEME CHANGES */
src/theme/Footer/Layout/enhanced-footer.css:1
- This block appears to have accidental edits:
line-height: 1;andborder-radius: 10px;were replaced by duplicatecolordeclarations, and there is nodisplayset on the badge (sojustify-contentlikely won’t apply). Restore the intended layout properties (e.g.,display: inline-flex,line-height,border-radius) and remove the duplicatecolorto prevent badge misalignment/shape regressions.
/* Enhanced Footer Styles - COMPLETELY ISOLATED FROM THEME CHANGES */
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| .btn { | ||
| display: inline-flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| padding: 10px 18px; | ||
| border-radius: 12px; | ||
| font-weight: 700; | ||
| text-decoration: none; | ||
| transition: transform 0.15s ease, box-shadow 0.15s ease; | ||
| white-space: nowrap; | ||
| } |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
✅ Synchronized metadata from Issue #1684:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Description
This PR redesigns footer of website as per attachments/reference given.
Fixes #1684
Type of Change
Changes Made
Checklist
npm run buildand attached screenshot(s) in this PR.