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

The CSS Working Group just discussed `element.isvisible`, and agreed to the following:

* `RESOLVED: Rename to checkVisibility()`

<details><summary>The full IRC log of that discussion</summary>
&lt;TabAtkins> Topic: element.isvisible<br>
&lt;TabAtkins> github: https://github.com/w3c/csswg-drafts/issues/7317<br>
&lt;TabAtkins> chrishtr: I think w'ere ready for the name. We had a vote up for the last week<br>
&lt;bkardell_> q+<br>
&lt;TabAtkins> chrishtr: 9 votes for checkVisibility() and only a few on anythign else, suggest we resolve on that<br>
&lt;TabAtkins> astearns: Small number of votes but it was the clear winner<br>
&lt;astearns> ack bkardell_<br>
&lt;TabAtkins> bkardell_: Don't want to block, but all feel like less-bad rather than good<br>
&lt;TabAtkins> bkardell_: I gave some ideas in the thread but it doesn't seem to have gone anywhere<br>
&lt;TabAtkins> bkardell_: Seems worth to take at least one more beat to be sure we consider that<br>
&lt;TabAtkins> astearns: That said, we've gone over this several times and solicited better names several times<br>
&lt;TabAtkins> astearns: Your suggestions in particular I'm not fond of due to spelling<br>
&lt;TabAtkins> astearns: Would think we'd've found a better term in the previous discussions if one existed<br>
&lt;bramus> q+<br>
&lt;astearns> ack bramus<br>
&lt;TabAtkins> bramus: If checkVisibility() is th ename, what does it return?<br>
&lt;TabAtkins> chrishtr: Still true/false, true if visible<br>
&lt;TabAtkins> fantasai: Agree with bkardell that this name isn't amazing, but think it's the best we have, and it doesn't have quite as much confusing downside as isVisible/Hidden. Fine with going for it since we don't have a better option and it's not too overloaded of a term.<br>
&lt;TabAtkins> astearns: Suppose we could resolve this as "best of everything we have so far", leaving it open for new suggestions but closing it to all the ones we've considered.<br>
&lt;TabAtkins> chrishtr: I think we can just resolve.<br>
&lt;TabAtkins> +1 to just resolving, fine with this name<br>
&lt;TabAtkins> florian: Also not huge fan of the name but found others problematic.<br>
&lt;TabAtkins> florian: This one might not be lovely but it doesn't cause problems<br>
&lt;TabAtkins> astearns: brian did you want to block?<br>
&lt;TabAtkins> bkardell_: Do not, it just didn't get much discussion.<br>
&lt;TabAtkins> bkardell_: Like you said, we need to make progress. If people think we won't do better, I support that.<br>
&lt;TabAtkins> astearns: So proposed resolution is checkVisibility()<br>
&lt;TabAtkins> RESOLVED: Rename to checkVisibility()<br>
</details>


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


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

Received on Wednesday, 29 June 2022 16:23:49 UTC