This is the D7 variant for issue #3152569: Allow image size to be configured. .

In short, this feature:

  • Adds a setting image size to the block settings form.
  • Adds the value of this setting to the js data that is used to initialize the library.
  • This setting does work in the sense that the dimensions of the downloaded images are changed.
  • However, the image sizes as displayed are calculated by the library, UNLESS you set the styling setting to false!
CommentFileSizeAuthor
#2 allow-image-size-to-be-configured-D7.patch2.73 KBawasson
sif-3152569.patch2.47 KBfietserwin

Comments

fietserwin created an issue. See original summary.

awasson’s picture

awasson commented
StatusFileSize
new allow-image-size-to-be-configured-D7.patch2.73 KB

I've updated the Drupal 8 DEV version with this feature so I though I'd check back on this one.

The patch in #1 doesn't work because it doesn't actually apply the image size variable to the feed when it instantiates the feed. I've re-rolled it for the most recent version and updated the code to use the image size variable. I've attache the latest patch.

It seems to work well. I'll move it to fixed shortly after I push the changes to DEV.

Cheers,
Andrew

  • awasson committed a4501b6 on 7.x-1.x authored by fietserwin 
    Issue #3160728 by fietserwin, awasson: Allow image size to be configured...
awasson’s picture

awasson commented
Status: Needs review » Fixed

Done. I'll close this as fixed.

@fietserwin take the latest dev for a spin and if you have any concerns, please reopen the issue. I'm going to jump on back porting the Lazy Loading of images but I'm happy to revisit this if we need to.

Cheers,
Andrew

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.