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

Magento 2.2.0 Solution for Cross-sell product placeholder image size … #12018

Merged
merged 2 commits into from Nov 7, 2017
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 4, 2017

Description

Magento 2.2.0 Solution for Cross-sell product placeholder image size …

Fixed Issues (if relevant)

  1. Cross-sell product placeholder image size issue #12017: Cross-sell product placeholder image size …

Manual testing scenarios

  1. Check the issues Cross-sell product placeholder image size issue #12017 for more detail.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@vkublytskyi vkublytskyi self-assigned this Nov 4, 2017
@vkublytskyi vkublytskyi added this to the November 2017 milestone Nov 4, 2017
<image id="cart_cross_sell_products" type="thumbnail">
<width>200</width>
<height>248</height>
<image id="cart_cross_sell_products" type="small_image">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes make sense. As Luma theme is developed mostly for demo but not for extending I would ask you to make similar changes for Blank theme that developed to be a parent for custom themes.

@ghost
Copy link
Author

ghost commented Nov 6, 2017

Hello @vkublytskyi

I have made similar changes for the Blank theme. Please review it from your end and let me know if we need any other changes on this.

@magento-team magento-team merged commit aa1c530 into magento:2.2-develop Nov 7, 2017
magento-team pushed a commit that referenced this pull request Nov 7, 2017
magento-team pushed a commit that referenced this pull request Nov 7, 2017
[EngCom] Public Pull Requests - 2.2-develop
 - MAGETWO-83279: Magento 2.2.0 Solution for Cross-sell product placeholder image size … #12018
 - MAGETWO-83270: 11700: "Something Went Wrong" error for limited access admin user #11993
 - MAGETWO-83085: fix #8846: avoid duplicated attribute option values #11785
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants