fix: add check for null hamburgerButton and sidebar elements #3802
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
User description
👋 Hi there! This PR was automatically generated by Autofix 🤖
This fix was triggered by DB
Fixes BLT-JAVASCRIPT-40-8M
hamburgerButton
andsidebar
elements exist before attaching the event listener.If you have any questions or feedback for the Sentry team about this fix, please email [email protected] with the Run ID: 7620.
PR Type
Bug fix
Description
Added null checks for
hamburgerButton
andsidebar
elements.Logged error message if required elements are missing.
Updated
sidebar
element to include anid
attribute.Changes walkthrough 📝
sidenav.html
Add null checks and error handling in sidebar script
website/templates/includes/sidenav.html
id
attribute to thesidebar
element.hamburgerButton
andsidebar
.