Ingvar Stepanyan
Ingvar Stepanyan
I think summary card would be a good start for now. Please do, especially if it doesn't take much time! We can always upgrade to screenshots later.
Yeah I'd start with `description` + `image` (V8 logo) + canonical URL. We can add more things later as necessary, but I think this would be enough for the start.
Was this fixed by https://github.com/v8/v8.dev/pull/365?
Looks like simply removing `height: auto` fixes the problem, but not 100% sure if there are any unwanted side-effects from doing that?
You seem to be referring to the entire block, but I was suggesting only removing `height:auto` part. Removing it and keeping `max-width` seems to result in correct aspect-ratio and dimensions...
@mathiasbynens That seems to be doing the opposite, as far as I can tell - if you look at e.g. the 4th image, its specified dimensions are ``. With `height:...
@tomayac You're looking at the first image, which, as you can see from the calculated image in issue description, is correct right away anyway. This is because that image is...
> It's calculating the aspect ratio based on width and height, leaves exactly the space it needs given the current actual width, and that's it. It's really not, and that's...
> however, when I comment out height: auto, then it's getting downsized with correct aspect-ratio to 764x336 even before the image is loaded, which seems like the desired behaviour? Nvm,...
Please sign the CLA. I'm surprised the bot didn't kick in, but you should be able to do so here https://cla.developers.google.com/.