Use direct element reference when updating render container

[grant-cleary]

This was still generating some console errors under some circumstances when an html-block was relatively quickly rendered and then updated again in quick succession. Here's what I think is happening: Before our async renderers can act, Lit is wiping out the value of the ref during its re-render. This leads to NREs in various places, since our renderers also do a bunch of asnyc things, so this can cause failures in a number of places.

While I could do something like const renderContainer = this._renderContainerRef.value to snag a direct reference to the element instead, I also think that throws away basically all of the value of using the ref directive anyways. 🤷

Let me know if you would find it preferable for me to switch to the above, but for now I think it makes more sense to just use a querySelector.

[github-actions]

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL	https://live.d2l.dev/prs/BrightspaceUI/core/pr-5047/

Note

The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

[d2l-github-release-tokens]

🎉 This PR is included in version 3.49.1 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀