Unexpected aspect ratio format

[tremby]

Currently, you have aspect ratio is set to the height as a percentage of the width.

Embed/src/EmbedCode.php

Line 23 in 8ac2150

$this->ratio = round(($height / $width) * 100, 3);

This isn't what most people would call aspect ratio; it should be width divided by height. (And not as a percentage.)

In fact, in the docs you actually say it's width divided by height:

Embed/README.md

Line 61 in 8ac2150

$info->code->ratio; //The aspect ratio (width/height)

(I noticed the unexpected value downstream here spicywebau/craft-embedded-assets#247 where I point out it'd be a breaking change, but something to consider for a major release.)

[oscarotero]

The ratio property was added before CSS supported aspect-ratio property. The purpose was to replicate it using the padding hack (more info here: https://nikitahl.com/css-aspect-ratio)

Now that CSS supports aspect-ratio, probably this property doesn't make sense because you can do simply:

$css_aspect_ratio = "$code->width / $code->height";

[tremby]

Regardless of its original purpose, and of the capabilities of CSS, and of whether you agree that most developers would be surprised to receive the current value, the logic used and the value returned do not agree with what is promised in the documentation.

[oscarotero]

Fair enough. I have edited the README.md file to improve the explanation for this property.