Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Entity Brand image transform does not meet Logo (Organization) structured data requirements #1372

Open
jonnykates opened this issue Oct 27, 2023 · 2 comments
Labels

Comments

@jonnykates
Copy link

jonnykates commented Oct 27, 2023

Describe the bug

An image asset uploaded to Site Settings > Identity > Entity Brand will be used in the Organization structured data. Google's documentation seems to suggest that the image needs to be "112x112px at minimum", but it looks like SEOmatic is generating a 60x60px transformation for these images. This only looks like a 'soft' requirement, in that the current 60x60px image still passes Search Console and Rich Results Test checks.

To reproduce

Steps to reproduce the behaviour:

  1. Set an image asset in the 'Site Settings > Identity > Entity Brand' field.
  2. The logo ImageObject will have height and width values set to 60
  3. The image in the url field will be of a 60x60px image
  4. Bonus quest: It looks in the URL params of the transformed image, that it is perhaps requesting w=600&h=60. Here's an example web page. The logo.url property is set to https://pura-aventura.transforms.svdcdn.com/production/uploads/images/partners/pura-logo-large-2020-oliveoutofwhite.png?w=600&h=60&q=82&fm=png&fit=clip&dm=1683754518&s=a0c9df0ab63fb4cb9a2cbced5ee4f3ff.

Expected behaviour

The logo object should have a width and height attribute of at least 112, and the image transformation should be for a 112x112px dimension.

Versions

  • Plugin version: 4.0.33
  • Craft version: 4.5.9
@jonnykates jonnykates added the bug label Oct 27, 2023
@khalwat
Copy link
Collaborator

khalwat commented Oct 30, 2023

So Google originally held that this brand image should be no larger than 600x60 pixels for the identity brand, presumably because they were using it in some kind of "banner" style in the SERP. That's why the transform is being done.

I'll keep this issue open as I research what the new current requirements are for this, and where the original reference originated.

@jonnykates
Copy link
Author

Ah OK interesting, so the 600x60 isn't a typo; apologies!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants