Skip to content
This repository has been archived by the owner on Jul 2, 2021. It is now read-only.

Pins with multiple images only end up using the thumbnail #114

Open
scytherswings opened this issue Jun 1, 2018 · 4 comments
Open

Pins with multiple images only end up using the thumbnail #114

scytherswings opened this issue Jun 1, 2018 · 4 comments
Labels
bug Something isn't working

Comments

@scytherswings
Copy link
Contributor

Right now this pin: https://app-prod.pinster.io/pin/93eb444e-4a02-49c5-b692-0afe0cca9216 When viewed in the modal on the front page only seems to use thumbnails. The expected behavior is that there's one thumbnail and two high res images when you look at it in the modal.

@scytherswings scytherswings added the bug Something isn't working label Jun 1, 2018
@markmoser
Copy link
Contributor

Confused. I found that pin in the listing and the modal has two images that toggle in the view as expected. Are you saying the images in the modal do not have high enough resolution?

@scytherswings
Copy link
Contributor Author

scytherswings commented Jun 5, 2018

So the main image container in the modal is showing this image for me:

<img class="pin-modal-img" src="http://pinster-image-service-prod.s3-website-us-east-1.amazonaws.com/bc7e008…_400x400">

I expected to see:

<img class="pin-modal-img" src="http://pinster-image-service-prod.s3-website-us-east-1.amazonaws.com/bc7e008…_1000x1000">

This honestly may have been fixed by your srcthing changes in dev and we should reapproach this once the prod push is finished.

@markmoser
Copy link
Contributor

related #117 #63 #110

@scytherswings
Copy link
Contributor Author

So this doesn't completely answer the question here, but I wrote a component for the native app that handles image service images. Check it out for some ideas maybe? https://github.com/IlluminusLimited/CoreUI/blob/develop/src/components/ImageServiceImage.js

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants