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).


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