Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v1.0.0b] - Sidebar always starts open on mobile reload #1090

Closed
5 tasks done
hay-kot opened this issue Mar 23, 2022 · 5 comments · Fixed by #1787
Closed
5 tasks done

[v1.0.0b] - Sidebar always starts open on mobile reload #1090

hay-kot opened this issue Mar 23, 2022 · 5 comments · Fixed by #1787
Labels
bug: confirmed bug Something isn't working good first issue Good for newcomers v1 Version 1 Issue/PR

Comments

@hay-kot
Copy link
Collaborator

hay-kot commented Mar 23, 2022

First Check

  • This is not a feature request
  • I added a very descriptive title to this issue.
  • I used the GitHub search to find a similar issue and didn't find it.
  • I searched the Mealie documentation, with the integrated search.
  • I already read the docs and didn't find an answer.

What is the issue you are experiencing?

When loading the application on mobile or as a PWA on a mobile device the sidebar always defaults to open. It should default to closed.

Deployment

Docker (Linux)

Deployment Details

No response

@hay-kot hay-kot added bug Something isn't working good first issue Good for newcomers v1 Version 1 Issue/PR labels Mar 23, 2022
@zierbeek
Copy link
Contributor

I thought it was a feature 😅

@zierbeek
Copy link
Contributor

This would have to do with #906 ?

@Miroito
Copy link
Contributor

Miroito commented May 9, 2022

I've started using Mealie on mobile since I deployed it a few days ago and I must admit I like that the sidebar is opened when you load the site.
What is not great and can probably be fixed is that sometimes clicking on a link in the sidebar gets you to that page but the sidebar doesn't disappear by itself.

@stale
Copy link

stale bot commented Jul 8, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 8, 2022
@dorj222
Copy link
Contributor

dorj222 commented Oct 23, 2022

if this issue still has not been solved, may I claim this issue?

hay-kot added a commit that referenced this issue Nov 11, 2022
* updated the sidebar; on mobile devices, the sidebar will be closed by default

* updated the AppSideBar

* change variable name

Co-authored-by: Hayden <[email protected]>
@hay-kot hay-kot moved this to Done in Mealie Development Nov 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: confirmed bug Something isn't working good first issue Good for newcomers v1 Version 1 Issue/PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants