Skip to content
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

Missing image in autocomplete variant #2951

Closed
sechix opened this issue Nov 14, 2018 · 5 comments · Fixed by #3032
Closed

Missing image in autocomplete variant #2951

sechix opened this issue Nov 14, 2018 · 5 comments · Fixed by #3032

Comments

@sechix
Copy link

sechix commented Nov 14, 2018

Trying to add a product variant to an order from admin. If variant don't have images no image appears. It should display master image as in the other parts of solidus.
image

In this case this can be solved modifying this file : 'assets->javascript->spree->backend->templates->variants->autocomplete.hbs' buy I have detected more handlebars using the same logic.

@jacobherrington
Copy link
Contributor

In Slack @vzqzac brought up:

wouldn’t show the master image misinform the user of the variant being selected? Thinking a bit about UX in here :thinking_face: open to discussion though

I think he has a point, but I also don't expect the people using this page to really understand the difference between a normal variant and a master variant. As long as the SKU is correct, I don't think it would be a bad user experience.

@sechix
Copy link
Author

sechix commented Nov 14, 2018

For example, in our case we use the same images always for all variants, so never assign the images to any variant. So if I am looking for a product I have to see the image associated in this case the master because don't have a specific image. This is the normal behavior in solidus. And don't disturb the other case, only add functionality.

@vzqzac
Copy link
Contributor

vzqzac commented Nov 14, 2018

In the projects I've worked on we have different images for each variant, so I think that showing the master variant's image could lead the user to think they're choosing that variant.

But I think @jacobherrington has a point too, users that can use the admin panel, and add products to orders, might have better knowledge of the application flow and thus rely on SKUs to make sure it's the variant they want.

Still, I think that relying on the master variant could have some UX issues

@sechix
Copy link
Author

sechix commented Nov 15, 2018

@vzqzac Only when variant don't show image has to display master image. This is how hows in every part of solidus

@vzqzac
Copy link
Contributor

vzqzac commented Nov 27, 2018

Right @sechix, we should keep consistency in this case in my opinion, if the way it is showed in this part changes, it should come along with many other places updated, so might be better to keep it as is in the rest of the project

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants