-
Notifications
You must be signed in to change notification settings - Fork 106
Replace markdown images with HTML img tags #358
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
Replace markdown images with HTML img tags #358
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this with browser dev tools and it's a massive improvement. Thank you!!
edit to add: Approved but the branch needs a rebase
Updated image embedding in performance.md from markdown syntax to HTML <img> tags for better control over attributes such as alt text and width.
ae67cdd to
98f1c98
Compare
|
I tested this using the browser dev tools, and it’s a massive improvement. Everything works great — this PR is good to go and ready to merge. |
|
The WordPress.org issue still exists. Has this PR been released? @kittenkamala |
Ugh! Thank you. The issue persists for me as well despite this being merged. |
I couldn't see the inline CSS width="100%" in the browser, so this makes the issue still persist |
#354
Updated image embedding in performance.md from markdown syntax to HTML
tags for better control over attributes such as alt text and width.