Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Detached elements node list is not populated when elements are fetched #233

Closed
TaranTwomey opened this issue Mar 1, 2024 · 14 comments
Closed
Labels
bug Something isn't working tracked This issue is now tracked on our internal backlog

Comments

@TaranTwomey
Copy link

TaranTwomey commented Mar 1, 2024

Recently I have been making great use of the 'Detached Elements' tool that Edge DevTools provides, however I noticed after a recent upgrade to the latest versions of Edge that it has stopped populating the Nodes list when collecting.

This issue appears to affect even the latest Canary, as well as the latest stable 122 version (122.0.2365.63) for 64-bit Windows 10. A coworker of mine who is also using the tool is still running 121.0.2277.128 and the tool appears to function correctly. I am currently unable to downgrade my version of edge, so I cannot test this precisly, but if I am able to downgrade, I will update this issue.

Reproduction:

  1. Load the demo page from microsoftedge.github.io/Demos/detached-elements/

  2. Following the same steps from the demo, generate traffic using the 'fast traffic' button.
    Generated Traffic, before tool is run
    image

  3. Switch to room 2.

  4. Run the detached elements tool by clicking 'get detached elements'.
    Switch Room, collect detached elements
    image
    Note that the count in the top right of the tool has updated to show the number of detached elements, but the node list is not populated

This issue appears to affect the latest Canary as well (124.0.2435.0 canary (64-bit)).

AB#49247073

@TaranTwomey TaranTwomey added the bug Something isn't working label Mar 1, 2024
@captainbrosset
Copy link
Contributor

Thanks so much for reporting this, and providing these details. I had not yet updated my Stable Edge, so I was able to test with 121 and confirmed that it worked fine there.
After update, the tool no longer displays the detached nodes.
I also tested in Canary (124), where the feature also doesn't work anymore.

So you've discovered a 122 regression that, unfortunately, went through our release process undetected. I'll mark this up for the team to investigate quickly.

@captainbrosset captainbrosset added the tracked This issue is now tracked on our internal backlog label Mar 1, 2024
@q837477816
Copy link

Same issue with Version 122.0.2365.80 (Official build) (x86_64) MacOS. Any updates?

@captainbrosset
Copy link
Contributor

No updates to share at this stage yet, but we're working on it.

@nyfer
Copy link

nyfer commented Mar 12, 2024

@captainbrosset till then any alternative that we can try?

@captainbrosset
Copy link
Contributor

You can also use the Memory tool and search for detached elements.

  1. Open the Memory tool.
  2. Run your scenario of choice on the webpage.
  3. In the tool, click Take snapshot to capture a heap snapshot.
  4. In the snapshot, type detached in the Class filter text box at the top.

image

Hope this helps.

@kheos31
Copy link

kheos31 commented Apr 23, 2024

Hello, still no update on this issue?

Thanks in advance.

@captainbrosset
Copy link
Contributor

No update to share yet on this. I'll get back to you all as soon as I hear more.

@j-sawtooth
Copy link

this would also be helpful for us to have working again.

@Chrisyjs
Copy link

Chrisyjs commented May 7, 2024

Hello, still no update on this issue?

@robpaveza
Copy link
Contributor

Hi @TaranTwomey and @Chrisyjs - this issue has been fixed in Edge 126. We don't currently plan to back-port this into 124 or 125 given our current release schedule. Edge 126 is currently in Canary and Dev channels, which you can download here: https://www.microsoftedgeinsider.com/

Thanks!

@tomeronen
Copy link

@robpaveza this work for me after update! Love this tool thanks so much ❤️

@captainbrosset
Copy link
Contributor

This is great, thank you for confirming that the feature is now working for you.
Closing the issue.

@alltabs
Copy link

alltabs commented May 15, 2024 via email

@alltabs
Copy link

alltabs commented May 15, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tracked This issue is now tracked on our internal backlog
Projects
None yet
Development

No branches or pull requests

10 participants