w3c / IntersectionObserver

Intersection Observer
https://www.w3.org/TR/intersection-observer/
Other
3.62k stars 523 forks source link

Should be "thresholdIndex" and not "threshold" in "Assign threshold to intersectionObserverRegistration’s previousThresholdIndex property." #332

Closed amn closed 2 years ago

amn commented 5 years ago

It seems to me that the correct wording of the sentence should be instead:

"Assign thresholdIndex to intersectionObserverRegistration’s previousThresholdIndex property."

There is no threshold definition in the list of steps for the algorithm outlined in section 3.2.5, and assigning a threshold to a variable that by all accounts is designed to store a threshold index, further points to an omission or a simple spelling error here.

miketaylr commented 2 years ago

It seems like this was fixed, the current language states, "Assign thresholdIndex to intersectionObserverRegistration’s previousThresholdIndex property." (in https://w3c.github.io/IntersectionObserver/#update-intersection-observations-algo).

Thanks for the issue.