Skip to content

[16.0][IMP] shopfloor_mobile: show product image in product details#1203

Open
nicolas-delbovier-acsone wants to merge 1 commit into
OCA:16.0from
acsone:16.0-imp-shopfloor-product-image-in-product-details
Open

[16.0][IMP] shopfloor_mobile: show product image in product details#1203
nicolas-delbovier-acsone wants to merge 1 commit into
OCA:16.0from
acsone:16.0-imp-shopfloor-product-image-in-product-details

Conversation

@nicolas-delbovier-acsone

@nicolas-delbovier-acsone nicolas-delbovier-acsone commented Jun 30, 2026

Copy link
Copy Markdown

Split part from: #1200

image

cc @jbaudoux

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @simahawk,
some modules you are maintaining are being modified, check this out!

@jbaudoux jbaudoux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not fan of having the picture inside the box. I find it renders better after the box like it was done in the cluster picking

Image

Comment thread shopfloor_mobile/static/wms/src/components/detail/detail_product.js Outdated
max-width="128"
aspect-ratio="1"
contain
class="rounded elevation-1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why rounded?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A purely aesthetic choice

@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-imp-shopfloor-product-image-in-product-details branch from 02f7537 to f3a1484 Compare June 30, 2026 13:19
@nicolas-delbovier-acsone nicolas-delbovier-acsone changed the title [16.0][IMP] shopfloor_mobile: show product imgae in product details [16.0][IMP] shopfloor_mobile: show product image in product details Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants