New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 617393 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

IntersectionObserver: console warning for non-descendant targets?

Reported by l446240525@gmail.com, Jun 4 2016

Issue description

<body>
  <div id=root></div>
  <div id=target></div>
</body>

<script>
new IntersectionObserver(callback, {root: root}).observe(target) // a warning in the Console
</script>
 
Cc: rnimmagadda@chromium.org
Labels: Needs-Feedback
@l446240525: Could you please provide us the Steps to reproduce, Expected & Actual results to triage this issue from our side. 

Also, please provide us the sample file to repro this issue, which would help us in triaging it further.

Thank you.
@rnimmagadda see https://bugs.chromium.org/p/chromium/issues/detail?id=540528#c52 Dev team will look into this issue.
@l446240525: Do you want me to merge this issue into - 540528 ?
@rnimmagadda: No, please label this issue as a feature request.
Project Member

Comment 5 by sheriffbot@chromium.org, Jun 6 2016

Labels: -Needs-Feedback Needs-Review
Owner: rnimmagadda@chromium.org
Thank you for providing more feedback. Adding requester "rnimmagadda@chromium.org" for another review and adding "Needs-Review" label for tracking.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Components: Platform>DevTools
Labels: -Type-Bug Type-Feature
Owner: ----
Status: Untriaged (was: Unconfirmed)
Could someone from DevTools team please look into this feature request.

Thank you.

Comment 7 by l...@chromium.org, Jun 8 2016

Cc: mpb@chromium.org
mpb@, if you are implementing Interaction Observers, would adding a warning be something under your attention?

Comment 8 by mpb@chromium.org, Jun 8 2016

Cc: szager@chromium.org
+szager, who did the current IO implementation. 
I have a patch in-flight to add the warning:

https://codereview.chromium.org/2048853002/

Comment 11 by l...@chromium.org, Jun 9 2016

Cc: l...@chromium.org
Owner: szager@chromium.org
Status: Fixed (was: Untriaged)
Nice job, szager@!

Comment 12 by 523...@gmail.com, Feb 24 2017

Hello, I did not set the root elem, but the chrome still give this warning: target element is not a descendant of root, is this wrong? I did not understand.

Sign in to add a comment