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-max height #65

Closed
zerox12311 opened this issue Nov 11, 2017 · 5 comments
Closed

Menu-max height #65

zerox12311 opened this issue Nov 11, 2017 · 5 comments

Comments

@zerox12311
Copy link

Hi, I'm using this theme
This is really a good theme,but had some bug...
when i menu-item is too much, it will not show all,
I found the max-height of .menu-items.expanded class is lock on 300px
I tried to change the menu css but not have effect,please help me......

@BryanYin
Copy link

@nnixaa
I also met this issue, could you please give quick fix? I think a simple change to remove the max-height limitation will do.
thanks!

@zerox12311
Copy link
Author

@BryanYin
Hi , I fix this issue is force override the CSS style in index.html style tag with

.menu-items.expanded,
    [_nghost-c2] .menu-item>.menu-items.expanded {
      max-height: 900px;
    }

@BryanYin
Copy link

@zerox12311
Thank you! this works like a charm!

@ffreitasit
Copy link

Why the max-height??? O.o

@nnixaa
Copy link
Collaborator

nnixaa commented Dec 19, 2017

Hi @ffreitasit, max-height there is for animation porpuses. But of course this is not an excuse and this has to be fixed.

@nnixaa nnixaa added this to the rc.6 milestone Jan 15, 2018
@nnixaa nnixaa closed this as completed in a753bee Jan 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants