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

Can't close sidebar on mobile #2071

Closed
KristiyanTs opened this issue Dec 7, 2020 · 6 comments
Closed

Can't close sidebar on mobile #2071

KristiyanTs opened this issue Dec 7, 2020 · 6 comments
Labels
bug 🐛 Something isn't working high priority This has a high priority

Comments

@KristiyanTs
Copy link
Contributor

Describe the bug

On a mobile device, after you open the sidebar there's no way to close it other than restarting the page.

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://ethereum.org/en/
  2. Open sidebar from top-right hamburger menu
  3. Try to close it

Expected behavior

There's an X button to close

Smartphone:

  • Device: Pixel 3
  • OS: Android
  • Browser Chrome
  • Version latest
@KristiyanTs KristiyanTs added the bug 🐛 Something isn't working label Dec 7, 2020
@KristiyanTs KristiyanTs changed the title Can close sidebar on mobile Can't close sidebar on mobile Dec 7, 2020
@samajammin samajammin added the high priority This has a high priority label Dec 7, 2020
@samajammin
Copy link
Contributor

Thanks for reporting this @Kristiyan96! Will investigate ASAP.

@samajammin
Copy link
Contributor

@all-contributors please add @Kristiyan96 for bug reports.

@allcontributors
Copy link
Contributor

@samajammin

I've put up a pull request to add @Kristiyan96! 🎉

@wackerow
Copy link
Member

wackerow commented Dec 7, 2020

@samajammin Looking at this now... the button got removed when the "mobile search" got added

@samajammin
Copy link
Contributor

Solved in #2074 (thanks @wackerow)!

Curious what you folks think of this interaction in the case that

  1. user opens mobile menu
  2. user opens search
  3. user closes search
    Should the menu stay open (as it does now?) Or should both close?

Here's a video to demonstrate:
https://share.getcloudapp.com/OAuWq6q7

@wackerow
Copy link
Member

wackerow commented Dec 7, 2020

@samajammin Personally, it makes sense the way it is. With the current animation design, when using search from the mobile menu, they visually layer on top of one another in two separate animations, which I would expect the search panel to unfold independently when closed (especially as the layout of the X changes slightly, which indicates it's a new/separate panel to a degree).
As-is, this only happens when navigating to search from the menu, and it offers a "go back" button that would not otherwise exist.
If you go straight to Search from the top nav bar however you wouldn't expect it to "go back" to a menu you never opened, and it doesn't.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working high priority This has a high priority
Projects
None yet
Development

No branches or pull requests

3 participants