Re: [csswg-drafts] [cssom-view] Rename Element.isVisible to Element.isHidden? (#7317)

To me, this isn't just about the name but what information the function is providing.

`isVisible`: We check a fixed, not necessarily comprehensive, set of conditions (opacity is 0, visibility is not visible, etc) and if none of those conditions are true, then we seemingly jump to the conclusion that the element must be visible -> return true
`isHidden`: We check the same fixed set of conditions and if none of those conditions are true, we say that _as far as we know_ this element is not hidden, but we're not promising that it's visible -> return false

Am I reading too much into the name?


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


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

Received on Friday, 3 June 2022 03:55:51 UTC