-
Notifications
You must be signed in to change notification settings - Fork 882
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
Twitter card size setting is broken #7719
Comments
@PatrickYoast @jcomack could you look into this and specify what we should be doing? |
Is being investigated in https://github.com/Yoast/roadmap/issues/24 |
@hedgefield Can you still reproduce this? When I switch the setting between |
Sorry, I should have specified: I mean the preview card we show in our metabox in WP. Changing the setting doesn't have any effect on that. |
Good to know! In that case, I do agree that we need to show a preview that's as closely related to the end result as possible. Whether that's a |
SO technically this isn't a bug, but an enhancement. At the same time, it's worth queueing. |
Could you add this to the roadmap @omarreiss ? |
It's a bug in the sense that it always shows the small card when no info has been entered and always shows the large card when you do have something filled in for the social preview/meta description, even though both templates exist and we have a setting to switch between them 🙃 |
Just to clarify: Basically, the last iteration of the Social Previews automatically detected the image size and changed the preview accordingly. The setting in the backend was never included in this check and therefore the preview could be considered broken. It might be a good idea to see what Twitter does if that option is omitted from the meta tags and possibly add some kind of "Auto-detect" option as well, depending on the result. |
In the plugin settings you can choose whether you want to use the small or large twitter card, but this setting is useless.
I expect it to use the format I chose, and at least have the blank and filled formats be consistent with each other.
The text was updated successfully, but these errors were encountered: