Re: [csswg-drafts] [css-shapes] Allow optional rounding parameter for `polygon()` (#9843)

> Would it prevent this problem if we clamp to sqrt(min(side1, side2))?

I don't see the reasoning of that formula. In particular sqrt makes values between 0 and 1 larger, making it easier to be problematic.

> One question to decide is whether we clamp all circles to the smallest of the radii (this is what border-radius does) or if each clamping operation is independent.

I'm leaning towards clamping independently

> I'm not certain how that [clamping the position of the center of the circle] would help

When I wrote the comment it seemed simpler to think in those terms to "ensure it can't go past the halfway point of its adjacent sides". But thinking again, using tan() to express the tangent point limits in terms of the radius is more straightforward.

Basically, the provided radius needs to be clamped to not exceed `tan(α/2) s₁ / 2` nor `tan(α/2) s₂ / 2`.

-- 
GitHub Notification of comment by Loirooriol
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/9843#issuecomment-1921430490 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Thursday, 1 February 2024 14:14:12 UTC