diff options
3 files changed, 14 insertions, 5 deletions
diff --git a/addons/mail/static/src/components/attachment/attachment.js b/addons/mail/static/src/components/attachment/attachment.js index a1c05fb8c62..a4b7b1363fe 100644 --- a/addons/mail/static/src/components/attachment/attachment.js +++ b/addons/mail/static/src/components/attachment/attachment.js @@ -94,10 +94,18 @@ class Attachment extends Component { if (this.detailsMode === 'card') { size = '38x38'; } else { - size = '160x160'; + // The size of background-image depends on the props.imageSize + // to sync with width and height of `.o_Attachment_image`. + if (this.props.imageSize === "large") { + size = '400x400'; + } else if (this.props.imageSize === "medium") { + size = '200x200'; + } else if (this.props.imageSize === "small") { + size = '100x100'; + } } // background-size set to override value from `o_image` which makes small image stretched - return `background-image:url(/web/image/${this.attachment.id}/${size}/?crop=true); background-size: auto;`; + return `background-image:url(/web/image/${this.attachment.id}/${size}); background-size: auto;`; } //-------------------------------------------------------------------------- diff --git a/addons/mail/static/src/components/attachment/attachment.scss b/addons/mail/static/src/components/attachment/attachment.scss index 22ff24f3880..583e5703691 100644 --- a/addons/mail/static/src/components/attachment/attachment.scss +++ b/addons/mail/static/src/components/attachment/attachment.scss @@ -71,7 +71,8 @@ &.o-details-overlay { position: relative; - + // small, medium and large size styles should be sync with + // the size of the background-image and `.o_Attachment_image`. &.o-small { min-width: 100px; min-height: 100px; diff --git a/addons/mail/static/src/components/attachment/attachment_tests.js b/addons/mail/static/src/components/attachment/attachment_tests.js index 302f39cd41a..eaeb267dfd8 100644 --- a/addons/mail/static/src/components/attachment/attachment_tests.js +++ b/addons/mail/static/src/components/attachment/attachment_tests.js @@ -107,8 +107,8 @@ QUnit.test('simplest layout + deletable', async function (assert) { async mockRPC(route, args) { if (route.includes('web/image/750')) { assert.ok( - route.includes('/160x160'), - "should fetch image with 160x160 pixels ratio"); + route.includes('/200x200'), + "should fetch image with 200x200 pixels ratio"); assert.step('fetch_image'); } return this._super(...arguments); |