elm-geometry icon indicating copy to clipboard operation
elm-geometry copied to clipboard

Add BoundingBox2d.aspectRatio

Open MartinSStewart opened this issue 4 years ago • 1 comments

I've found that I've needed this function a couple times. Writing the code in a one-off fashion looks like this

( w, h ) =
    BoundingBox2d.dimensions boundingBox

aspectRatio =
    Quantity.ratio h w

safeAspectRatio = 
    if isNaN aspectRatio || isInfinite aspectRatio then
        Nothing
    else
        Just aspectRatio

This is verbose so it would be nice to just have as BoundingBox2d.aspectRatio: BoundingBox2d -> Maybe Float and possibly BoundingBox2d.aspectRatioInv: BoundingBox2d -> Maybe Float for the width over height version.

Edit: I know elm-geometry tends to avoid having functions that depend on a particular x y coordinate system but I think an exception should be made for BoundingBox2d since it's already defined in terms of an x y coordinate system.

MartinSStewart avatar May 30 '20 07:05 MartinSStewart

I think I'd be inclined to just return a Float, to be consistent with Quantity.ratio and other Elm functions in general (sqrt, the built-in division operator etc.) that can return infinity or NaN.

ianmackenzie avatar Oct 02 '20 00:10 ianmackenzie