Inserting images wider than editor viewport causes horizontal scroll bar about roosterjs HOT 3 CLOSED

It is from

Instead of using view port width, we can use client width of editor. @juliaroldi can you take a look? BTW, can you remind me why we change from 100% to view port width?

from roosterjs.

It is from

roosterjs/packages/roosterjs-content-model-core/lib/coreApi/formatContentModel/formatContentModel.ts

Line 107 in 25f0296

const viewport = core.api.getVisibleViewport(core);

Instead of using view port width, we can use client width of editor. @juliaroldi can you take a look? BTW, can you remind me why we change from 100% to view port width?

I think it is related to this change https://github.com/microsoft/roosterjs/pull/2126/files. If the image is too tall, it is not possible to resize or crop the image. Since we are not using this code for adjusting the height, should adjust the image with 100% width?

from roosterjs.

It is from

roosterjs/packages/roosterjs-content-model-core/lib/coreApi/formatContentModel/formatContentModel.ts

Line 107 in 25f0296

const viewport = core.api.getVisibleViewport(core);

Instead of using view port width, we can use client width of editor. @juliaroldi can you take a look? BTW, can you remind me why we change from 100% to view port width?

I think it is related to this change https://github.com/microsoft/roosterjs/pull/2126/files. If the image is too tall, it is not possible to resize or crop the image. Since we are not using this code for adjusting the height, should adjust the image with 100% width?

We used 100% before, but for some reason we changed it. Will the original problem come back if we go back to 100%?

from roosterjs.