Display additional images below primary product image

[amrarick26]

F51-386

Display the default image along with additional images set on the product.xp when in the product detail view.

product.xp : {
  image: {
    URL: 'defaultimageURL'
  },
  additionalImages: [
    {
      image: {
        URL: 'additionalImageURL1'
      }
    },
    {
      image: {
        URL: 'additionalImageURL2'
      }
    }
  ]
}

[robertsoniv]

Please make the changes described in my comment on the productDetail.images.html file

[robertsoniv]

@amrarick26 You should do your ngRepeat over a new array variable that combines the default image (product.xp.image) with the additionalImages array. Make the default image the first item in the array so that it is active by default.

This new array is also what should be passed into the product images modal to get the same result there.

[amrarick26]

@robertsoniv that is being done in the productDetail controller.

[robertsoniv]

@amrarick26 @spencerwalker @crhistianr Do y'all think it would make sense to roll the default image display (when there are no additional images) into the oc-product-images directive? That way we just have to move that one directive around when coming up with new PDT layouts.

[robertsoniv]

@amrarick26 would you like to update this PR with the changes we talked about? Or just submit a new one once you have time to do it.

Basically, display a single image with a lightbox if there aren't any additional images and image zoom is turned on.

[robertsoniv]

If image zoom is off and there are no additional images, just show the thumbnail of the default image

[robertsoniv]

If there are no images... placeholder still? Or just don't show anything? @spencerwalker @crhistianr

[amrarick26]

@robertsoniv doesn't that seem kind of redundant if they only have one image, to show another thumbnail version of that image?

[amrarick26]

@robertsoniv here