Uploaded image for project: 'Imaging'
  1. Imaging
  2. MGNLIMG-212

Don't scale up small images and make it configurable

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Unresolved
    • Neutral
    • None
    • 3.4.2
    • None
    • Yes
    • Yes

    Description

      Follow SUPPORT-9771,

      If we upload a small image (as example 80x80) and the content will use an image variation like "large" with a max-width of 1200, the image will be scaled up from 80x80 to 1200x1200. This will destroy the quality of image.

      After investigated, I have found that setting 'expand' property to 'false' will not expand your image according to our implementation below:
      https://git.magnolia-cms.com/projects/MODULES/repos/imaging/browse/magnolia-imaging/src/main/java/info/magnolia/imaging/operations/cropresize/BoundedResize.java#48

      However it is not documented yet. Also if users are using ThemeAwareImageGenerator, this 'expand' property is not configurable from its SimpleResizeVariation.

      Expected result:

      • Set info.magnolia.imaging.operations.cropresize.BoundedResize.expand to false by default
      • Document its usage
      • Make it configurable for ThemeAwareImageGenerator somehow

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              Unassigned Unassigned
              viet.nguyen Viet Nguyen
              Votes:
              1 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:

                Checklists

                  Task DoD