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

Updated index.md with icons #296

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

Conversation

ArjunGK22
Copy link

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.

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:

/kind new feature
/kind bug fix
/kind cleanup
/kind revert change
/kind design
/kind documentation
/kind enhancement

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?:

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind TESTED
/kind NOT-TESTED

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

Issue sodafoundation#83

Added Icon for Twitter / GitHub / Linked etc. icons on Introduction page.
Copy link
Member

@anvithks anvithks left a 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.

- [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">&nbsp; [SODA Website](https://sodafoundation.io/the-foundation/join/)
Copy link
Member

Choose a reason for hiding this comment

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

Please check the deploy preview.
The logos are not appearing.
image

Copy link
Author

@ArjunGK22 ArjunGK22 Apr 4, 2023

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

content/introduction/_index.md Outdated Show resolved Hide resolved
@anvithks
Copy link
Member

anvithks commented Apr 6, 2023

@ArjunGK22 Please check the deploy preview. Looks like the alignment is not correct.
image

Consider a different approach to making this section. You need not just add the logos to the list.
The primary intention of adding logos was to make it more easier for people to identify the service / platform that they could reach us on.

@ArjunGK22
Copy link
Author

Is there any way to look at the deployment preview before giving a pull request. Becoz when I see locally it is perfect.

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.

2 participants