joshwnj/react-visibility-sensor

Evaluate IntersectionObserver support

Open

#56 建立於 2016年12月16日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)JavaScript (2,300 star) (198 fork)batch import
enhancementhelp wanted

描述

Suggested by @Andarist in https://github.com/joshwnj/react-visibility-sensor/pull/54#issuecomment-267279074

What would you say about implementing IntersectionObserver support? It could even be the default with scroll and interval being used only as a fallback settings. Its already implemented in 2 browsers.

https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API

It would be cool to try this, and compare to the current approach. We can try the polyfill for browsers that don't support it yet:

https://github.com/WICG/IntersectionObserver/tree/gh-pages/polyfill

http://caniuse.com/#search=Intersection

貢獻者指南

技術棧
reactjavascript
領域
frontendperformance
議題類型
feature
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
half day
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
stale
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
mostly clear
前置要求
React knowledgeIntersectionObserver APIPolyfill understanding
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
50
研究方向
This issue involves implementing IntersectionObserver support in the react visibility sensor component. The idea was suggested in PR #54 (linked). To start, review the current component source code (likely in src/index.js or similar) and understand how it currently uses scroll/interval events. Then implement an IntersectionObserver based approach as the default, with scroll/interval fallback for unsupported browsers. The polyfill from WICG can be used for older browsers. Test the changes and compare performance with the current approach. Check the caniuse link for browser support. There is no linked PR, so this is open for contribution.