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

In the img tag, the alt and closed are missing. #659

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laxman1192
Copy link

@laxman1192 laxman1192 commented Aug 28, 2020

On this "wp-includes/SimplePie/Parser.php" file I found the "img" tag you have used but the tag is not closed on end as well as the "alt" attribute is not used on the same.
As a standard workflow of HTML, The alt attribute and closing (< />) is required for the img element.

On this "wp-includes/SimplePie/Parser.php" file I found the "img" tag you have used but the tag is not closed on end as well as the "alt" attribute is not used on the same.
As a standard workflow of HTML, The alt attribute and closing (< />) is required for the img element.
@skyzyx
Copy link
Member

skyzyx commented Aug 28, 2020

@laxman1192 said:

On this "wp-includes/SimplePie/Parser.php" file I found the "img" tag you have used but the tag is not closed on end as well as the "alt" attribute is not used on the same.
As a standard workflow of HTML, The alt attribute and closing (< />) is required for the img element.

This statement is untrue. I added some links and information in the issue you closed. #658 (comment)

laxman1192 added a commit to laxman1192/simplepie that referenced this pull request Aug 30, 2020
Added alt with value and close on img tag on Parser.php file.
simplepie#658
simplepie#659

Thanks,
Laxman P
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants