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

Jump to bottom

Need to escape alt tag #1049

Closed
MohammedKaludi opened this issue Aug 13, 2017 · 1 comment
Closed

Need to escape alt tag #1049

MohammedKaludi opened this issue Aug 13, 2017 · 1 comment
Assignees
@MARQAS
Milestone
0.9.59

Comments

@MohammedKaludi
Copy link
Collaborator

alt tag needs escaping @MARQAS use esc_attr($var_name);

User is facing the validation issue because of this. Ref: https://wordpress.org/support/topic/trying-to-get-rid-of-one-final-small-error-to-get-our-amp-homepage-working/
@MohammedKaludi MohammedKaludi added this to the 0.9.59 milestone Aug 13, 2017
@Frenchomatic Frenchomatic mentioned this issue Aug 14, 2017
Closed
MARQAS added a commit that referenced this issue Aug 17, 2017
@MARQAS
#1049 #1050 Add Alt tags properly
f4c4f93
MARQAS added a commit that referenced this issue Aug 18, 2017
@MARQAS
#1049 #1050 Proper escaping of atts in alt tags
b206a96
MARQAS added a commit that referenced this issue Aug 18, 2017
@MARQAS
#1049 #1050
c9b8f9c 
Forgot to select these lines while committing(my bad 😐 )
@ahmedkaludi
Copy link
Owner

Reviewed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MARQAS MARQAS

Labels
None yet
Projects
None yet
Milestone
0.9.59
Development

No branches or pull requests
3 participants
@MARQAS @ahmedkaludi @MohammedKaludi