-
Notifications
You must be signed in to change notification settings - Fork 226
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
BUG: Found a Bug in MJML Image Block #339
Comments
Hi @RakulAgn, auto is not a valid value to mj-image, if you remove the width property of mj-image it will be set to 100%. |
Yeah thats right but its not handled in grapesjs mjml. in user point of view what if they type auto incase ? |
Thanks for finding this. Here we have a custom build of mjml, so we have solved :D Also, i've made a PR to it. #344 |
Thanks dude it took 2 month :( |
Description:
When attempting to set the height and width of an image to "auto" in the Grapesjs MJML Example on the official site, an error is thrown in the Chrome console. Once this error occurs, the image cannot be cloned unless it is completely removed.
### Steps to Reproduce:
Expected Behavior:
The image's height and width should adjust to "auto" without any errors. Cloning functionality should remain unaffected.
Actual Behavior:
An error is thrown in the console when setting the image's height and width to "auto". Cloning functionality for the image is disabled until the image is completely removed.
The text was updated successfully, but these errors were encountered: