Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upImprove: Responsive Horizontal-to-Vertical Menu #495
Comments
|
Thanks for the report and suggestion. I'll re-examine this when I next release the pure-site. It's possible that I started with auto but discovered a certain browser was having trouble unless I explicitly set the height, so keep browser vagaries in mind. |
|
I recently found out this gets worse when you have more menu items - they get cut off after the 14em. |
|
@inexist3nce @qskousen only problem with adding |
|
Wondering if this ties into my issue where child-items in the vertical version of the nav don't open on hover. Can be seen here in this CodePen. Or is this expected behavior and I need to handle showing child items on my own in the vertical version of the menu? The following code seems to solve my issue, as long as it overrides the default yui-pure-css styling:
Also found the need to add Seems like I'm either doing something wrong, or the base case could be more intuitive. |
The responsive menu at http://purecss.io/layouts/tucked-menu-vertical/ does not adjust height as one would expect.
When viewing in a screen smaller than 768px, the "responsive" menu has a fixed height that could potentially cause issues with long menus (or in really small screens with wrapping text).
This can be fixed by replacing the hardcoded
height: 14em;at line 69 withheight: auto;.