We can split onvisibility change and send impression handling inside impression wrapper, essentially writing a visibility wrapper. sendImpression method can be defined outside of visibility wrapper and can be passed to VisibilityWrapper from outside e.g. <VisibilityWrapper onVisible={sendImpression} /> We should pick this implementation after the UI requirements for 76 are finalized.
Bug 1619694 Comment 2 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
We can split onvisibility change and send impression handling inside impression wrapper, essentially writing a visibility wrapper. sendImpression can be passed to VisibilityWrapper from outside e.g. <VisibilityWrapper onVisible={sendImpression} /> We should pick this implementation after the UI requirements for 76 are finalized.