Skip to content

Conversation

Riya-Kharade
Copy link
Contributor

Which issue does this PR close?

Rationale for this change

The issue was raised because social media icons in the footer were reported as barely visible against the light background.
However, after checking the latest fork of the repo, the social media icons are no longer present.
The main visibility problem was actually with the footer logo, which was white and not clearly visible on the background.

What changes are included in this PR?

  • Updated footer logo to improve visibility on the background
  • Adjusted spacing in the footer for a cleaner look

Are these changes tested?

  • Verified manually in browser (desktop + mobile view)
  • Checked responsive breakpoints for layout consistency

Screenshots

When the PR was first raised (icons visible):
image

When I forked the repo today (icons no longer present):
image

After my changes (logo updated):
image

Copy link

netlify bot commented Sep 20, 2025

Deploy Preview for growcraftt ready!

Name Link
🔨 Latest commit 5432929
🔍 Latest deploy log https://app.netlify.com/projects/growcraftt/deploys/68ce84f67eb17800084c4737
😎 Deploy Preview https://deploy-preview-577--growcraftt.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@gyanshankar1708 gyanshankar1708 merged commit f331ae5 into gyanshankar1708:master Sep 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix Footer Icons Visibility & Spacing
2 participants