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

footer added #1225

Merged
merged 9 commits into from
Aug 11, 2023
Merged

footer added #1225

merged 9 commits into from
Aug 11, 2023

Conversation

BhartiNagpure
Copy link
Contributor

Fixed #1209

Close #1209

In this PR I'm added footer into the footer pages.
Screenshot 2023-08-09 214906

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you BhartiNagpure! for creating this pull request and contributing to Dummygram! 💗

The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀

@BhartiNagpure
Copy link
Contributor Author

@narayan954 done with PR please review it.

Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

Hi @BhartiNagpure , footer component is quite confusing name, can you change it to a better name, like footerpagefooter or something better.
Also, due to some recent changes, there's a little need to update it.. can you please do it??
image
remove those extra icons on footer and hence make it small, one liner, with just those copyright and all lines you had added. Also, give it a background color with variable to make it compatible with theme

@BhartiNagpure
Copy link
Contributor Author

Screenshot 2023-08-11 154125

@narayan954 here is updated footer as you required changes please review it.

Copy link
Owner

@narayan954 narayan954 left a comment

Choose a reason for hiding this comment

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

lgtm!

@narayan954 narayan954 merged commit 2932e4c into narayan954:master Aug 11, 2023
3 of 4 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.

[FEATURE] footer
2 participants