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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated the footer design #105

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

kulendu
Copy link

@kulendu kulendu commented Aug 27, 2020

Hello,
I have modified the footer design. Added a Back to top button, which was previously a love link and also added the footer logo.
Also the button takes to the top section of the page.

Before:

After:

Hope it would help 馃憤

@kulendu
Copy link
Author

kulendu commented Aug 29, 2020

Hey @jancborchardt, please review the changes that I have made.
Please review the PR
馃憤

@jancborchardt
Copy link
Member

Hm I kind of like the little heart, the "Back to top" link can be separate though? And I don鈥檛 think a logo in the footer is necessary. :) What do you think @jonatoni?

@kulendu sorry for the very late reply!

@jonatoni
Copy link
Member

Actually I like the little heart too. 馃槃 And I agree we don't need the logo in the footer

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.

None yet

3 participants