Fix: Issue #88 - Distorted images on stores

[vsseixaso]

What problem is this solving?

This PR aims to fix the Issue #88. An issue generated in release v0.22.0, which allowed setting the image height of an image-list via Admin Site Editor, but set maxHeight as default for the image's height prop. This distorted the image in accounts that had a value set for maxHeight.

With this PR, the image height will no longer receive maxHeight as the default value, requiring the user to set a value for the prop via Admin Site Editor.

How to test it?

Access the Site Editor and follow this sequence of blocks:

HEADER > Header-Desktop > Slider Layout then click on Image List.

Then, access any image and set a value in the Height field, example: 300px.
Save and apply, now you can see the height of the image is different, but defined by your change, and not by default.

Screenshots or example usage:

Before

After

Describe alternatives you've considered, if any.

Follow up: Add an option to set height for all images in the image-list, for now it's necessary set one by one.

How does this PR make you feel? 🔗

[vtex-io-ci-cd]

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

• Patch (backwards-compatible bug fixes)

• Minor (backwards-compatible functionality)

• Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

• No thanks, I would rather do it manually 😞

[vtex-io-docs-bot]

Beep boop 🤖

I noticed you didn't make any changes at the docs/ folder

• There's nothing new to document 🤔
• I'll do it later 😞

In order to keep track, I'll create an issue if you decide now is not a good time

• I just updated 🎉🎉

[github-actions]

Fails

🚫

We follow keepachangelog standards. Please, add a section to your changelog entry. Example: ## [Unreleased] + ### Fixed - Issue with image sizing in the SKU Selector. You can use one of the following sections: Added for new features. Changed for changes in existing functionality. Deprecated for soon-to-be removed features. Removed for now removed features. Fixed for any bug fixes. Security in case of vulnerabilities.

Generated by 🚫 dangerJS against 56053b7

[vmourac-vtex]

LGTM

[iago1501]

Nice catch

[vtex-io-ci-cd]

Your PR has been merged! App is being published. 🚀
Version 0.22.0 → 0.22.1

After the publishing process has been completed (check #vtex-io-releases) and doing A/B tests with the new version, you can deploy your release by running:

vtex deploy [email protected]

After that your app will be updated on all accounts.

For more information on the deployment process check the docs. 📖