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

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