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

Hamburger menu close button refinements #44

Closed
tomayac opened this issue Dec 13, 2021 · 7 comments
Closed

Hamburger menu close button refinements #44

tomayac opened this issue Dec 13, 2021 · 7 comments
Assignees

Comments

@tomayac
Copy link
Owner

tomayac commented Dec 13, 2021

  • It gets rendered outside of the menu on iOS.
  • It should remain at a fixed position when scrolling.
@tomayac
Copy link
Owner Author

tomayac commented Dec 13, 2021

Hello @georapbox! Are you interested in working on this?

@georapbox
Copy link
Contributor

georapbox commented Dec 13, 2021

Hi @tomayac! I'll have a look when I find some time, hopefully today!

@tomayac
Copy link
Owner Author

tomayac commented Dec 13, 2021

Absolutely no pressure, take whatever time you need, or just say "no", too.

@tomayac
Copy link
Owner Author

tomayac commented Dec 13, 2021

The iOS issue:

image

@georapbox
Copy link
Contributor

Regarding having the button in fixed position, there are might be some edge cases that the button might fall over other elements. This can happen in very small viewports in combination to other languages, eg Greek where the words are longer. The minimum I can do is to provide a background color to button so that at least it does not look bad if that happens. Would that be a viable solution? Otherwise, I guess that we would need to follow another approach, like wrapping the button in another fixed positioned element with a background color that would act as a menu header and push the menu's content a few pixels lower.

@tomayac
Copy link
Owner Author

tomayac commented Dec 13, 2021

I guess in the interest of screen real estate, adding a background color and making the button float on top would work best here.

@tomayac
Copy link
Owner Author

tomayac commented Dec 13, 2021

Fixed via #45

@tomayac tomayac closed this as completed Dec 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants