Skip to content

Fix IE11 error in Shopping Cart #8108

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

Closed
wants to merge 2 commits into from
Closed

Conversation

Sanfam
Copy link

@Sanfam Sanfam commented Jan 11, 2017

The checkout attempts to call a resize event that IE11 cannot use. This breaks shipping rate calculation on this page. While it doesn't prevent a user from moving forward, the effect is misleading and implies that they have stalled.

If they're using IE11 in 2017, odds are good that this is enough to deter them from proceeding any further.

Bringing repo up to date
This browser may only account for <10% of all traffic on many sites, but that's still a statistically significant quantity.
@Sanfam Sanfam changed the title Make Checkout IE11 Compatible Fix IE11 error in Shopping Cart Jan 11, 2017
@antonkril antonkril requested a review from que-etc January 11, 2017 17:21
@vasiliyseleznev vasiliyseleznev requested review from omiroshnichenko and removed request for que-etc January 20, 2017 10:15
@omiroshnichenko
Copy link
Contributor

Hi, @Sanfam.
I have check cart and checkout functionality without your fix(on develop). I can`t receive any errors. Also, in past, I have fix issue that you describe in this commit. Can you check again and provide more information? Thanks.

@vrann
Copy link
Contributor

vrann commented Mar 1, 2017

@Sanfam thanks for the contribution! I'm closing it now, but if you still can reproduce the issue please open another PR.

@vrann vrann closed this Mar 1, 2017
@vrann vrann added this to the March 2017 milestone Mar 1, 2017
@vrann vrann self-assigned this Mar 1, 2017
magento-devops-reposync-svc pushed a commit that referenced this pull request Feb 6, 2023
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.

5 participants