-
Notifications
You must be signed in to change notification settings - Fork 220
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
Updated index.md with icons #296
base: master
Are you sure you want to change the base?
Conversation
Issue sodafoundation#83 Added Icon for Twitter / GitHub / Linked etc. icons on Introduction page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check the review comments. I have linked the Learn Hugo theme documentation. This is the theme we are using for the website.
content/introduction/_index.md
Outdated
- [SODA Twitter](https://twitter.com/sodafoundation) | ||
- [SODA Mailing List](https://lists.sodafoundation.io) | ||
- [SODA LinkedIn](https://www.linkedin.com/company/sodafoundation/) | ||
- <img src="https://github.com/FortAwesome/Font-Awesome/blob/6.x/svgs/brands/github.svg" width="20" height="20"> [SODA Website](https://sodafoundation.io/the-foundation/join/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure sir...I will check and update it....Thank you
@ArjunGK22 Please check the deploy preview. Looks like the alignment is not correct. Consider a different approach to making this section. You need not just add the logos to the list. |
Is there any way to look at the deployment preview before giving a pull request. Becoz when I see locally it is perfect. |
Issue #83
Added Icon for Twitter / GitHub / Linked etc. icons on Introduction page.
What type of PR is this?
This PR adds the icons to the links like github, mail, linkedin, slack in the introduction page.
What this PR does / why we need it:
Gives a visual navigation to user rather than just textual.
Which issue(s) this PR fixes:
Fixes #83
Test Report Added?:
Test Report:
Special notes for your reviewer:
@anvithks Kindly have a look at it and let me know if any changes has to be made. Thank you