-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Advanced-Search-layout-not-proper #21611
Advanced-Search-layout-not-proper #21611
Conversation
Hi @amol2jcommerce. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
@magento-engcom-team give me test instance |
Hi @rogyar. Thank you for your request. I'm working on Magento instance for you |
Hi @rogyar, here is your new Magento instance. |
Hi @amol2jcommerce. Thank you for your collaboration. It's looks better now on desktop resolutions. However, if you take a look at iPad (portrait) resolution - the form is quite small. The contact us form has the same issue, so it's not the best example. However, if you check the create account form - it behaves much better in terms of responsive design. Try to compare the result of your solution and the create account form on different resolutions. |
Hi @rogyar, I updated branch please review. |
@magento-engcom-team give me test instance |
Hi @sanganinamrata. Thank you for your request. I'm working on Magento instance for you |
Hi @sanganinamrata, here is your new Magento instance. |
Hi @rogyar, I updated branch as per suggestion please review. |
@magento-engcom-team give me test instance |
Hi @rogyar. Thank you for your request. I'm working on Magento instance for you |
Hi @rogyar, here is your new Magento instance. |
Hi @rogyar, thank you for the review. |
✔️ QA Passed |
Hi @VladimirZaets, thank you for the review. |
Hi @amol2jcommerce, thank you for your contribution! |
Description (*)
Advanced Search layout not proper
Fixed Issues (if relevant)
Manual testing scenarios (*)
Contribution checklist (*)