W3C home > Mailing lists > Public > public-css-archive@w3.org > September 2017

Re: [csswg-drafts] [cssom-view] Add notIfViewed to ScrollIntoViewOptions; add FocusScrollOptions

From: Simon Pieters via GitHub <sysbot+gh@w3.org>
Date: Tue, 12 Sep 2017 09:52:35 +0000
To: public-css-archive@w3.org
Message-ID: <issue_comment.created-328803111-1505209945-sysbot+gh@w3.org>
Note that this introduces a boolean dictionary member with default value `true`, which is generally not recommended.

> It is strongly suggested not to use a default value of true for boolean-typed arguments, as this can be confusing for authors who might otherwise expect the default conversion of undefined to be used (i.e., false).

https://heycam.github.io/webidl/#dfn-optional-argument-default-value

However, the alternative would be to use different members that both default to `false` but have different semantics (i.e., `notIfViewed = false` and `evenIfViewed = false`), which also seems like it would cause confusion. cc @tobie

-- 
GitHub Notification of comment by zcorpan
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/pull/1805#issuecomment-328803111 using your GitHub account
Received on Tuesday, 12 September 2017 09:52:29 UTC

This archive was generated by hypermail 2.4.0 : Tuesday, 5 July 2022 06:41:17 UTC