Skip to content

Commit 885a800

Browse files
review nits
1 parent fc4040f commit 885a800

File tree

1 file changed

+15
-11
lines changed

1 file changed

+15
-11
lines changed

index.bs

+15-11
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ urlPrefix: https://html.spec.whatwg.org/multipage/
4949
type: dfn; text: origin
5050
urlPrefix: https://heycam.github.io/webidl/
5151
url: #dfn-simple-exception; type:exception;
52-
text: NotAllowedError
5352
text: RangeError
5453
text: TypeError
5554
text: SyntaxError
@@ -464,9 +463,8 @@ dictionary IntersectionObserverEntryInit {
464463
has zero area).
465464
: <dfn>isVisible</dfn>
466465
::
467-
True if {{IntersectionObserver/trackVisibility}} is <code>true</code>
468-
and the <a>visibility</a> algorithm,
469-
when run on {{IntersectionObserverEntry/target}}, returns <code>true</code>.
466+
Contains the result of running the <a>visibility</a> algorithm
467+
on {{IntersectionObserverEntry/target}}.
470468
: <dfn>intersectionRatio</dfn>
471469
::
472470
If the {{IntersectionObserverEntry/boundingClientRect}} has non-zero area,
@@ -603,6 +601,12 @@ IntersectionObserver</h4>
603601
which is a list of four pixel lengths or percentages.
604602
* A <dfn attribute for=IntersectionObserver>\[[scrollMargin]]</dfn> slot
605603
which is a list of four pixel lengths or percentages.
604+
* A <dfn attribute for=IntersectionObserver>\[[thresholds]]</dfn> slot
605+
which is initialized by {{IntersectionObserver(callback, options)}}.
606+
* A <dfn attribute for=IntersectionObserver>\[[delay]]</dfn> slot
607+
which is initialized by {{IntersectionObserver(callback, options)}}.
608+
* A <dfn attribute for=IntersectionObserver>\[[trackVisibility]]</dfn> slot
609+
which is initialized by {{IntersectionObserver(callback, options)}}.
606610

607611
<h3 id='algorithms'>
608612
Algorithms</h2>
@@ -632,10 +636,10 @@ and an {{IntersectionObserverInit}} dictionary |options|, run these steps:
632636
8. If |thresholds| is empty, append <code>0</code> to |thresholds|.
633637
9. The {{IntersectionObserver/thresholds}} attribute getter will return
634638
this sorted |thresholds| list.
635-
10. If |options|.{{IntersectionObserverInit/trackVisibility}} is <code>true</code>,
636-
and |options|.{{IntersectionObserverInit/delay}} is not specified or is less than <code>100</code>,
637-
<a>throw</a> a {{NotAllowedError}} exception.
638-
11. Set |this|'s internal {{[[delay]]}} slot to |options|.{{IntersectionObserverInit/delay}}.
639+
10. Let |delay| be the value of |options|.{{IntersectionObserverInit/delay}}.
640+
11. If |options|.{{IntersectionObserverInit/trackVisibility}} is true
641+
and |delay| is less than <code>100</code>, set |delay| to <code>100</code>.
642+
11. Set |this|'s internal {{[[delay]]}} slot to |options|.{{IntersectionObserverInit/delay}} to |delay|.
639643
12. Set |this|'s internal {{[[trackVisibility]]}} slot to |options|.{{IntersectionObserverInit/trackVisibility}}.
640644
13. Return |this|.
641645

@@ -650,8 +654,8 @@ and an {{Element}} |target|, follow these steps:
650654
an {{IntersectionObserverRegistration}} record
651655
with an {{IntersectionObserverRegistration/observer}} property set to |observer|,
652656
a {{IntersectionObserverRegistration/previousThresholdIndex}} property set to <code>-1</code>,
653-
a {{IntersectionObserverRegistration/previousIsIntersecting}} property set to <code>false</code>,
654-
and a {{IntersectionObserverRegistration/previousIsVisible}} property set to <code>false</code>.
657+
a {{IntersectionObserverRegistration/previousIsIntersecting}} property set to false,
658+
and a {{IntersectionObserverRegistration/previousIsVisible}} property set to false.
655659
3. Append |intersectionObserverRegistration|
656660
to |target|'s internal {{[[RegisteredIntersectionObservers]]}} slot.
657661
4. Add |target| to |observer|'s internal {{[[ObservationTargets]]}} slot.
@@ -749,7 +753,7 @@ run these steps:
749753
Compute whether a Target is unoccluded, untransformed, unfiltered, and opaque.</h4>
750754

751755
To compute the <dfn>visibility</dfn> of a <a for="IntersectionObserver">target</a>, run these steps:
752-
1. If the |observer|'s {{IntersectionObserver/trackVisibility}} attribute is false, return true.
756+
1. If the |observer|'s {{IntersectionObserver/trackVisibility}} attribute is false, return false.
753757
2. If the <a for="IntersectionObserver">target</a> has an <a>effective transformation matrix</a> other than a 2D translation or proportional 2D upscaling, return false.
754758
3. If the <a for="IntersectionObserver">target</a>, or any element in its <a>containing block chain</a>, has an effective opacity other than 100%, return false.
755759
4. If the <a for="IntersectionObserver">target</a>, or any element in its <a>containing block chain</a>, has any filters applied, return false.

0 commit comments

Comments
 (0)