Century21Stores / GLOBAL

Repository to archive any code and issues for enhancements, fixes, or new functionalities across the site (globally)
https://www.c21stores.com/
0 stars 0 forks source link

Size Sorting on PDP & Master Image Error Fix #22

Open c21hornak opened 8 years ago

c21hornak commented 8 years ago

Note: related to https://jira.tools.weblinc.com/browse/CENTURYROR-904 Note: related to https://jira.tools.weblinc.com/browse/CENTURYROR-556

c21hornak commented 8 years ago

In progress, about 80% complete

juanmriosv commented 8 years ago

You can find it on staging and production (more sizes options)

c21hornak commented 8 years ago

@AdenaG ready to QA (on PROD).

AdenaG commented 8 years ago

Fail in Prod & Staging:

  1. Sizes arent in order on Safari
  2. If an item has multiple colors, when you select a secondary color, the sizes aren't in order. screen shot 2016-08-23 at 12 32 12 pm
juanmriosv commented 8 years ago

@AdenaG Besides not working on Safari (have to look further into this, will take a bit longer) and when select multiple colors, everything else is fine?

@c21hornak

juanmriosv commented 8 years ago

@c21hornak Currently working on:

juanmriosv commented 8 years ago

@c21hornak @AdenaG Color selection issue has been resolved.

AdenaG commented 8 years ago

@juanmriosv Looks good, except I see mens pants sizes are still out of order. https://www.c21stores.com/products/signature-flat-front-straight-pants?color=Bradshaw&via=57e1465569702d03cf0000fa%2C57e1465669702d03cf000117

juanmriosv commented 8 years ago

@AdenaG will look into this. Still owe you the Safari issue

AdenaG commented 8 years ago

Sorry @juanmriosv: Now I am seeing the same issue on IE 11 as on Safari - the sizes aren't in order . Maybe my IE wasn't up to date when I first reviewed this.

c21hornak commented 7 years ago

@juanmriosv could you please expand the fix so that if the user clicks beyond the first color swatch, then the product image on the master, the error doesn't occur?

The YSL Scarf has lots of swatches so would be common for a customer to get the error on this Master. https://www.c21stores.com/products/ysl-scarf