Improved the UI of the footer section of the website#1517
Improved the UI of the footer section of the website#1517Shayan-Bhowmik wants to merge 1 commit into
Conversation
|
@Shayan-Bhowmik 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! 😊 |
There was a problem hiding this comment.
Pull request overview
This PR aims to polish the site footer's visual design by tweaking spacing, badge sizing, column-title prominence, and newsletter input/button touch targets in src/theme/Footer/Layout/enhanced-footer.css. However, in addition to the intended CSS tweaks, the file appears to have been run through a CSS formatter that flattened multi-line linear-gradient(...) and radial-gradient(...) calls and inserted blank lines inside @keyframes, and—more importantly—the diff has dropped closing braces around the responsive media-queries section, causing a large region of the stylesheet to become invalidly nested. The PR also includes unrelated package-lock.json churn that contradicts the PR description.
Changes:
- Footer styling tweaks: larger column titles with text-shadow, bigger badge/input/button paddings, increased link/section spacing, and full-width newsletter on mobile.
- File-wide reformatting of multi-line
linear-gradient/radial-gradientcalls and added blank lines inside keyframes. - Unrelated
package-lock.jsonmodifications (peer flags added/removed, two sub-entries deleted).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/theme/Footer/Layout/enhanced-footer.css | Footer spacing/sizing improvements plus reformatting; introduces broken brace nesting in the responsive section. |
| package-lock.json | Unrelated lockfile churn that conflicts with the PR description. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| .newsletter-column { | ||
| grid-column: span 2; | ||
| } |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
@Shayan-Bhowmik , can you into this build error , suggesting unclose brackets and something . |
|
@Adez017 yes looking into it |
Description
Fixes #1495
UI/UX improvements to the site footer based on screenshot feedback: increased spacing between links, made column titles more prominent, standardized badge sizing, increased newsletter input / CTA touch targets, added extra padding between stats and footer links, and ensured the newsletter box displays full-width and last on mobile.
Type of Change
Changes Made
CSS updates to the footer to improve layout and touch targets:
Updated
src/theme/Footer/Layout/enhanced-footer.css:Increased vertical spacing between items in
.footer-linksand added increasedmarginon.footer-link.Emphasized column headings via
.footer-column-titleusing:font-sizefont-weighttext-shadowStandardized badge sizing via
.link-badge:inline-flexIncreased
.newsletter-inputand.newsletter-buttonpadding and added responsive rules to make them full-width on small screens.Added extra vertical padding to
.footer-links-sectionand additionalmargin-bottomto.footer-statsfor improved separation between sections.Added responsive ordering fallback for
.newsletter-columnto ensure it appears last on stacked/mobile layouts.No structural changes to JSX/TSX were required;
src/theme/Footer/Layout/index.tsxalready places the newsletter column last.Dependencies
How to Test / Verify
1. Install dependencies and start the development server
Open the development site (usually
http://localhost:3000).2. Verify Desktop Layout
Check the following:
Learning,Company,Support,Stay in the Loop) appear slightly larger and more visually prominent.Popular,New,Hot,We're Hiring!) have consistent height and alignment.3. Verify Mobile Layout
Using browser DevTools responsive mode or a real device:
4. Optional Production Verification
Ensure the production build completes successfully without warnings or errors.
Checklist
npm run buildand attached screenshot(s) in this PR.