@mglaman Good catch on upscaling. When I remove that it works. Putting it back it breaks again, so it is the upscaling that causes the problem.
Confirming that I do not have any other effects in use. I'm testing with one at a time.
@shwetaDevkate The image tag itself has 500. The url is adding 8256.
@mglaman I inspected a little more. The 8256 is coming from the original image. I looked in Widen and the has its width listed as 8256. I tried embedding the same image twice. One with the scale image effect, and one using original image.
Using original image the tag adds h and w attributes with the sizes from Widen.
<img src="https://abc123.widen.net/content/ceaf09ae-c338-4556-82d2-84866e20b240/web/IMG.jpg" width="8256" height="5504" alt="IMG.jpg" loading="lazy">
The image display in this case because no width is added to the url as a parameter. When I open that url directly, Widen returns an image limited to 2048. So it appears to default to that as a max size.
When I use scale effect I get the tag reported, which puts width in the url. The width it gets from the source, so I something is causing it to use that value instead of the width from the effect (500.)
I have other people working on a different project that are also testing acquia_dam, and encountered the same problem, so I don't think it's my specific case.
If you are trying yourself, see if you can find an image in your Widen instance that is larger that 2048.
davidhernandez → created an issue.
@mglaman I am on php 8.1.
davidhernandez → created an issue.
davidhernandez → created an issue.
I'm using composer and this should be the latest dev release. It matches the commit id. I'll make a new issue.
"name": "drupal/acquia_dam",
"version": "dev-1.0.x",
"source": {
"type": "git",
"url": "https://git.drupalcode.org/project/acquia_dam.git",
"reference": "edca371760b6f7ed146bfee8e76a3e13aefb9fb2"Following up specifically about the form error. This patch resolves it. I'm seeing some other errors in the log that may be other issues. I'll try to verify and open other issues.
@mglaman I was partly mistaken. It looks like crop works and resize works, but the scale and crop image effect doesn't work. Is that supposed to be supported? I'm using basic image styles, view modes, in Drupal on an image field to display the image. So the urls generated in Drupal to retrieve the Widen asset.
Example:
<img src="https://abcdef.widen.net/content/484d0b1f-df73-4a3d-9ac4-88d7a65e5303/web/image.png?h=200&crop=yes&k=c&w=200&itok=K6aVmGmT" width="200" height="200" alt="CHIRP_wide.png" loading="lazy">
I'm playing with this field trying different things. It seems rotate works as long as it is 90deg, and I think desaturate worked, but I didn't realize it was specifically the scale and crop I was having issue with.
I can open a separate issue if this sounds broken.
@mglaman Thanks. I'll test this. Do you think this will resolve any issues with the link generation? That also seems to have an issue. The links generated are not using any crop functions from the image styles.
Not to open up a bikeshed, but it might be helpful if those that participated in the decision to prefer React provide some points as to why they didn't prefer Vue.