open-sauced / app

🍕 Insights into your entire open source ecosystem.
https://pizza.new
Apache License 2.0
381 stars 202 forks source link

feat: don't show sidebar by default for non-logged in user #3576

Closed zeucapua closed 2 weeks ago

zeucapua commented 2 weeks ago

Description

If the user is NOT logged in, the sidebar should not be showing by default. Vice versa if the user is logged in and in a large screen size.

Related Tickets & Documents

Closes #3573

Mobile & Desktop Screenshots/Recordings

Steps to QA

  1. Go to the deploy preview

  2. Ensure you are logged out AND you are in a large screen size

  3. Remove the showingSidebar variable from the local storage via browser tools

  4. Refresh and see the sidebar isn't showing

  5. Do the same with a logged in user and see that the sidebar shows

netlify[bot] commented 2 weeks ago

Deploy Preview for oss-insights ready!

Name Link
Latest commit 93de720c5f066cd7f8d1776cd6b9d82e77abc170
Latest deploy log https://app.netlify.com/sites/oss-insights/deploys/6671d85c0f96850008885bd4
Deploy Preview https://deploy-preview-3576--oss-insights.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 2 weeks ago

Deploy Preview for design-insights ready!

Name Link
Latest commit 93de720c5f066cd7f8d1776cd6b9d82e77abc170
Latest deploy log https://app.netlify.com/sites/design-insights/deploys/6671d85c96ba4000086242c4
Deploy Preview https://deploy-preview-3576--design-insights.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.