Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

issue: #4427; Added the Alternate text to image tags #4429

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

Conversation

sathiyaIbe
Copy link

Yeah it's my first contribution I had updated the images with missing alt : alternate text to the image tag So kindly check the changed files

…use alt text is important to understand the image
@welcome
Copy link

welcome bot commented May 24, 2023

💖 Thanks for opening this pull request! 💖

Here is a list of things that will help get it across the finish line:

  • Run yarn lint locally to catch formatting errors. This will fix some errors automatically, commit and push any changes.
  • Run yarn test locally to catch errors. This ensures all components still behave as they should.
  • Run yarn start to run the doc site locally and try a few pages, ensuring everything is in good working order.
  • Include tests when adding/changing behavior.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

@codecov-commenter
Copy link

codecov-commenter commented May 24, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (038b306) 99.51% compared to head (4400b7f) 99.51%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4429   +/-   ##
=======================================
  Coverage   99.51%   99.51%           
=======================================
  Files         186      186           
  Lines        3511     3511           
=======================================
  Hits         3494     3494           
  Misses         17       17           
Impacted Files Coverage Δ
src/modules/Embed/Embed.js 100.00% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants