Skip to content

Update footer with social icons #9

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

Merged
merged 6 commits into from
Sep 24, 2021

Conversation

toth2000
Copy link
Member

Description

Added social icons to the footer.

image

This PR fixes #2

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@welcome
Copy link

welcome bot commented Sep 23, 2021

Yay, your first pull request! 👍 A contributor will be by to give feedback soon. In the meantime, please review the Layer5 Contributors' Welcome Guide and sure to join the community Slack.
Be sure to double-check that you have signed your commits. Here are instructions for making signing an implicit activity while peforming a commit.

Signed-off-by: Tothagata Bhattacharjee <tothagata.bhattacharjee@gmail.com>

Signed-off-by: toth2000 <tothagata.bhattacharjee@gmail.com>
@toth2000 toth2000 force-pushed the toth2000/update_footer branch from 41b8d40 to 7891ebe Compare September 23, 2021 06:53
Copy link
Member

@debo19 debo19 left a comment

Choose a reason for hiding this comment

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

@toth2000 Looks good 🚀 , just some minor changes required here

<p align="center">Connect With Us</p>
<p align="center">
<a href="https://discuss.layer5.io/"><img alt="discourse_logo"
src="..\assets\social-icons\discourse.svg"></a>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
src="..\assets\social-icons\discourse.svg"></a>
src="https://github.com/layer5io/.github/blob/master/assets/social-icons/discourse.svg"></a>

src="..\assets\social-icons\discourse.svg"></a>
&nbsp;
<a href="mailto:community@layer5.io"><img alt="email_logo"
src="..\assets\social-icons\email.svg"></a>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
src="..\assets\social-icons\email.svg"></a>
src="https://github.com/layer5io/.github/blob/master/assets/social-icons/email.svg"></a>

Copy link
Member

Choose a reason for hiding this comment

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

Can you please change all the src to call the images globally like this, otherwise the images don't seem to load on the overview page if called locally.

Copy link
Member Author

Choose a reason for hiding this comment

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

@debo19 yes, I will make the suggested changes.

Signed-off-by: toth2000 <tothagata.bhattacharjee@gmail.com>
@toth2000 toth2000 requested a review from debo19 September 23, 2021 10:52
Copy link
Member

@debo19 debo19 left a comment

Choose a reason for hiding this comment

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

readmeSocials
@toth2000 Good work here. I think we can decrease this space between these two lines a little.
Also, while you're there, can you reduce the size of the icons slightly? This should be good to go after that.

@toth2000
Copy link
Member Author

toth2000 commented Sep 23, 2021

readmeSocials
@toth2000 Good work here. I think we can decrease this space between these two lines a little.

It will not be possible. But the actual gap will be less than the image that I have attached check this (as the image is from a VS Code markdown preview extension).

Also, while you're there, can you reduce the size of the icons slightly? This should be good to go after that.

Yes, I will make the changes.

@toth2000
Copy link
Member Author

Updated the icon's color for better visibility on dark mode.

image

image

toth2000 and others added 2 commits September 24, 2021 02:30
Signed-off-by: toth2000 <tothagata.bhattacharjee@gmail.com>
@toth2000 toth2000 requested a review from debo19 September 23, 2021 21:05
@leecalcote
Copy link
Member

Make sure to maintain aspect ration on these images. Currently, they are skewed.

@toth2000
Copy link
Member Author

Looking into it.

Signed-off-by: toth2000 <tothagata.bhattacharjee@gmail.com>
@toth2000
Copy link
Member Author

Made the suggested changes and the icons now looks like this:

image

Copy link
Member

@debo19 debo19 left a comment

Choose a reason for hiding this comment

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

LGTM 🎉 Thanks @toth2000

@debo19 debo19 merged commit d180833 into layer5io:master Sep 24, 2021
@welcome
Copy link

welcome bot commented Sep 24, 2021

Thanks for your contribution to the Layer5 community! 🎉

Congrats!
        Join the community, if you haven't yet and please leave a ⭐ star on the project. 😄

@leecalcote
Copy link
Member

@toth2000 you did some great work here. These footer icons really turned out well.

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.

[Markdown] Enhance footer with social icons
3 participants