We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
CSSMathClamp’s min and max attributes are incompatible with CSSNumericValue’s methods min and max.
CSSMathClamp
min
max
CSSNumericValue
This design violates the Liskov substitution principle. Also current version of the interface cannot be properly typed with TypeScript and Flow.
Possible compatible names for these attributes are lower and upper.
lower
upper
The text was updated successfully, but these errors were encountered:
Ah, indeed, that's a silly mistake on my part. It doesnt' just violate Liskov, it violates WebIDL!
Sorry, something went wrong.
is this going to be fixed? shouldn't have min and max as operation and attribute =/
operation
attribute
bd98097
Thanks for the ping! Fixed now.
No branches or pull requests
CSSMathClamp
’smin
andmax
attributes are incompatible withCSSNumericValue
’s methodsmin
andmax
.This design violates the Liskov substitution principle. Also current version of the interface cannot be properly typed with TypeScript and Flow.
Possible compatible names for these attributes are
lower
andupper
.The text was updated successfully, but these errors were encountered: