Microsoft Edge

[InteliClic]

Microsoft Edge

[perrytew]

Nick,
Still no solution. This issue has made it to our next sprint, so I'll have a solution by 2/5.
Sorry for the delay.

Internal Note: JIRA RESTAPI-5

[InteliClic]

Thanks for the update Perry. Definitely looking forward to a solution.

Currently showing edge users a notice letting them know we are incompatible, asking them to use another browser.

[perrytew]

Nick,
I've walked through this several times. The issue seems to be with the localstorage.

There's something I don't understand.
When the checkout page loads, you're loading the page from localstorage, a variable named ic-cart.

However, on the quick view page, after adding an item with the addItems() javascript method, where are you saving the cart back to localstorage?

The only place where the localstorage 'ic-cart' variable is set is from within the updateLocalCart() function, The only places that method is called is the login() method, within the document ready method, MyAccountWrapper.loggedIn,

I would expect to see a call to updateLocalCart made from within the addItems() methods between lines 910-911. Also, I don't like line line 910:

window.cart.items = updatedCart.items;

The notion of only updating part of a cart is a bad practice. The totals, subtotals, and all that jazz are updated constantly, especially when adding items to the cart. I would rather see:

window.cart = updatedCart;
updateLocalCart();

Thoughts?