Skip to content

Updated Footer Layout#180

Merged
naveen-bandela-au2 merged 2 commits intogofr-dev:mainfrom
sonaligupta08:update-footer
Sep 24, 2025
Merged

Updated Footer Layout#180
naveen-bandela-au2 merged 2 commits intogofr-dev:mainfrom
sonaligupta08:update-footer

Conversation

@sonaligupta08
Copy link

Adjusted spacing, layout, and styling in the footer to improve readability and consistency across screen sizes.

Copy link
Member

@coolwednesday coolwednesday left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a good practice, please open a GitHub issue first and wait for it to be assigned to you before starting any work. Submitting a PR without an associated issue or assignment can lead to it being closed or disregarded at the discretion of the maintainers.

Additionally, make sure to include a screenshot showing the change: "Before & After."

@sonaligupta08
Copy link
Author

sonaligupta08 commented Jul 7, 2025

Thank you for the feedback!
I’m still a beginner and learning the correct open-source workflow — I understand now that I should have created and been assigned to an issue first before submitting the PR. I’ll follow that process moving forward.

Also, I’ve attached the requested before and after screenshots below.

Before_&_after.pdf

@sonaligupta08
Copy link
Author

sonaligupta08 commented Jul 7, 2025 via email

@Umang01-hash
Copy link
Member

@sonaligupta08 The link to before and after is broken and also please create an issue for the changes you want to do.

Thank you for the feedback! I’m still a beginner and learning the correct open-source workflow — I understand now that I should have created and been assigned to an issue first before submitting the PR. I’ll follow that process moving forward.

Also, I’ve attached the requested before and after screenshots below.

Uploading Before_and_After.pdf.docx…

@sonaligupta08
Copy link
Author

Here is the corrected before and after screenshot in a PDF format.
before_and_after.pdf

Sorry for the earlier confusion

@naveen-bandela-au2 naveen-bandela-au2 merged commit d4ad11e into gofr-dev:main Sep 24, 2025
5 checks passed
@sonaligupta08 sonaligupta08 deleted the update-footer branch October 2, 2025 19:14
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.

4 participants