newrelic / instant-observability-website

Dashboards, alerts, and integrations all in one place. Our quickstarts bundle everything you need to start monitoring like a pro right out of the box.
https://newrelic.com/instant-observability/
Apache License 2.0
5 stars 20 forks source link

Changes to page pathname close mobile menu #259

Closed tabathadelane closed 2 years ago

tabathadelane commented 2 years ago

The header now detects when a user clicks the back button while the mobile menu overlay is open and closes it while navigating.

I chose to use the logic in the gatsby-theme-newrelic MobileNavigation component. I was having a lot of styling issues come up when trying to import the component directly.

I was seeing odd behavior in the local server where sometimes when clicking the hamburger icon, I would have to click it twice to get the overlay to open. I can't seem to reproduce it in a build, so please check this when reviewing.

gatsby-cloud[bot] commented 2 years ago

Gatsby Cloud Build Report

instant-observability-website

:tada: Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

:clock1: Build time: 59s

Performance

Lighthouse report

Metric Score
Performance :large_orange_diamond: 29
Accessibility :green_heart: 100
Best Practices :green_heart: 92
SEO :large_orange_diamond: 85

:link: View full report

nr-opensource-bot commented 2 years ago

:tada: This PR is included in version 1.2.2 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: