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

Update index.html #125

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

Conversation

2201monesh
Copy link

I have made the changes kindly review it.

I have made the changes kindly review it.
index.html Outdated
@@ -198,7 +198,7 @@ <h3>Common Voice</h3>
</a>
<a class="col-sm-6 col-md-4 p-4" href="https://chaoss.community/metrics/" target="_blank" rel="noopener">
<img alt="" src="img/projects/chaoss.jpg" />
<h3>CHAOSS D&I Workgroup</h3>
<h3>CHAOSS D Workgroup</h3>
Copy link
Member

Choose a reason for hiding this comment

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

It is D&I, could you please change the D to D&I?

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Author

Choose a reason for hiding this comment

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

@ramyaparimi changes are done, I am sorry for replying this late.

@@ -92,7 +92,7 @@
<h1 class="jumbotron-heading">Open Source Diversity</h1>
<p class="lead">For more diversity & inclusion in free & open source software communities 😊</p>
<p>
<a href="https://discourse.opensourcediversity.org/" class="btn btn-primary" target="_blank" rel="noopener"><img src="img/discourse.svg" /> Join our forum!</a>
<a href="https://discourse.opensourcediversity.org/" class="btn btn-primary" target="_blank" rel="noopener"><img alt="" src="img/discourse.svg" /> Join our forum!</a>
Copy link
Member

Choose a reason for hiding this comment

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

thank you @2201monesh for your contribution especially making the site more ally compliant, perhaps we can also add an alt text here rather than an empty string :)

Copy link
Member

Choose a reason for hiding this comment

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

@2201monesh are you still interested in working on this?

As mentioned the '&I' is added after G in line 201
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

4 participants