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

To summarize, the current options that we’ve seen are the following:

* isHidden - current name
 * May be confused with Element.hidden attribute
 * Tab’s twitter poll indicates that isVisible is more preferred
 * Semantically, it would have no false positives but may have false negatives
* isVisible
 * May be confused with IntersectionObserver’s isVisible
 * Preferred in Tab’s twitter poll
 * Semantically, it would have false positives, but no false negatives
* checkVisibility
 * Is not biased towards “is visible” or “is hidden”
 * Unclear about the return value: boolean may not be the best fit
 * Maybe return HiddenByX vs Unknown enums, which may be less ergonomic
* computeVisibility
 * Same arguments as checkVisibility, just different name

For posterity, Jen’s twitter poll seems to be mixed with roughly similar support for both isHidden and isVisible.

Are there other suggestions?


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


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

Received on Wednesday, 8 June 2022 17:16:06 UTC