blencorp/react-lighthouse-viewer

Container element for lighthouse report should be more specific

Closed this issue · 2 comments

calvinnwq commented

react-lighthouse-viewer/src/index.js

Line 44 in b2152ad

const container = document.querySelector("main");

This selector assumes that there are no other <main> elements on the page. If there is, especially one that is high up in the DOM tree, then it breaks.

naodya commented

@calvinnwq, thanks for pointing that out. On it.

naodya commented

@calvinnwq, <main> now has a unique class. Closing issue.