Skip to content

Commit ddaa3cc

Browse files
committed
✨(docx) fix image overflow by limiting width to 600px during export
ensures all images keep proportions and stay within page bounds in docx export Signed-off-by: Cyril <c.gromoff@gmail.com>
1 parent e5581e5 commit ddaa3cc

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ and this project adheres to
2222

2323
- 🐛(frontend) fix duplicate document entries in grid #1479
2424
- 🐛(backend) fix trashbin list
25+
- 🐛(docx) fix image overflow by limiting width to 600px during export #1525
2526
- ♿(frontend) improve accessibility:
2627
- ♿(frontend) remove empty alt on logo due to Axe a11y error #1516
2728
- 🐛(backend) fix s3 version_id validation

src/frontend/apps/impress/src/features/docs/doc-export/blocks-mapping/imageDocx.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,9 @@ export const blockMappingImageDocx: DocsExporterDocx['mappings']['blockMapping']
5050

5151
const { width, height } = dimensions;
5252

53-
if (previewWidth && previewWidth > MAX_WIDTH) {
54-
previewWidth = MAX_WIDTH;
55-
}
53+
// Ensure the final width never exceeds MAX_WIDTH to prevent images
54+
// from overflowing the page width in the exported document
55+
const finalWidth = Math.min(previewWidth || width, MAX_WIDTH);
5656

5757
return [
5858
new Paragraph({
@@ -71,8 +71,8 @@ export const blockMappingImageDocx: DocsExporterDocx['mappings']['blockMapping']
7171
}
7272
: undefined,
7373
transformation: {
74-
width: previewWidth || width,
75-
height: ((previewWidth || width) / width) * height,
74+
width: finalWidth,
75+
height: (finalWidth / width) * height,
7676
},
7777
}),
7878
],

0 commit comments

Comments
 (0)