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

Fixes (#72) : Refactor footer component and import it in Intro component #79

Closed
wants to merge 2 commits into from

Conversation

Venkateeshh
Copy link

fixes : #72
This pull request refactors the Footer component to enhance readability and maintainability, and integrates it into the Intro component, ensuring a consistent user interface across the application. Please review the changes.

Before

Screenshot 2024-10-27 222519

After

Screenshot 2024-10-27 222536

  • I have starred and forked the JobSewa repo.
  • I have carefully read the instructions for contributing to the project.
  • I have created a new page or section and placed it inside the appropriate folder.
  • My contributions align with the current project structure and conventions.
  • I have included any necessary images in a dedicated folder within assets, ensuring they are compressed and under 500kb.
  • All images are in .png format and have been directly uploaded—no external image links are used.
  • I understand that the maintainers will review my pull request and provide feedback if necessary.
  • I have assigned a reviewer to this pull request.

Copy link

vercel bot commented Oct 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
job-sewa ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 28, 2024 1:33pm

@Devmangrani
Copy link
Owner

Screenshot 2024-10-28 at 12 31 45 PM

Great work on the footer refactor—it's looking fantastic. Could you adjust the alignment on the homepage, though? The increased footer height has shifted the "Find Your Perfect Job with JobSewa" line up towards the navbar, which makes the main content feel a bit cramped. Changing this would really help the layout feel more balanced. I'll share a screenshot to illustrate. Thanks for your efforts!

@Venkateeshh
Copy link
Author

yea sure will work on it

@Venkateeshh
Copy link
Author

Screenshot 2024-10-28 at 12 31 45 PM Great work on the footer refactor—it's looking fantastic. Could you adjust the alignment on the homepage, though? The increased footer height has shifted the "Find Your Perfect Job with JobSewa" line up towards the navbar, which makes the main content feel a bit cramped. Changing this would really help the layout feel more balanced. I'll share a screenshot to illustrate. Thanks for your efforts!

fixed the issue

@Venkateeshh
Copy link
Author

i have fixed the issue mentioned in review

@Venkateeshh
Copy link
Author

can it be merged?

@Devmangrani
Copy link
Owner

no It is can't able to merge

@Venkateeshh Venkateeshh closed this Nov 7, 2024
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.

[FEATURE] : Improve website footer
2 participants