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

Menu not displaying on mobile for Headers UI components #37

Closed
lgaspari opened this issue Nov 2, 2022 · 3 comments
Closed

Menu not displaying on mobile for Headers UI components #37

lgaspari opened this issue Nov 2, 2022 · 3 comments

Comments

@lgaspari
Copy link

lgaspari commented Nov 2, 2022

Hey there! I'm new to Mantine and I was looking for a way to make my application's header, so I went right into https://ui.mantine.dev/category/headers.

The problem here is that even though the layout is responsive, the menu is not. For instance, if we look at https://ui.mantine.dev/component/header-menu-colored we can see that when narrowing the window the burger menu is displayed, but upon clicking it does nothing (see video below). Is this behavior intentional to keep it short? Otherwise, how could I display the menu on mobile?

Does anyone have a clue on that? I'm thinking that a <Menu> should do the trick, but would be nice to have that as part of the responsive example. Thanks!

Screen.Recording.2022-11-02.at.14.35.18.mov
@rtivital
Copy link
Member

rtivital commented Nov 2, 2022

We provide only one fully responsive header – https://ui.mantine.dev/component/header-responsive
You can use it as reference and build your components based on it

@lgaspari
Copy link
Author

lgaspari commented Nov 3, 2022

We provide only one fully responsive header – https://ui.mantine.dev/component/header-responsive You can use it as reference and build your components based on it

Hi @rtivital — Amazing, I wouldn't have found it by myself, thanks!

One side note though, IMO it would be nice to have this header example either at the top of the page or described in a way people could find it easier since it's right in the middle. Moreover, all headers are tagged as "responsive" but that one you sent is the only one that has "Responsive" on their name, so it feels a bit confusing.

I'm going to close this issue as it was resolved, but feel free to reopen if needed.

@lgaspari lgaspari closed this as completed Nov 3, 2022
@danmaz74
Copy link

danmaz74 commented Feb 6, 2023

@rtivital finding about the special importance of header-responsive is very difficult; a suggestion would be to add this information to the comments in the other examples. I was really confused until I found out about this. I could create a MR with this info added if you'd like to.

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

3 participants