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
[Drilldown Menu] Menu height is not dynamic #7821
Comments
This has been mentioned before, but there was never a clear idea as to what should happen on transition for a drill down. It felt far to jarring to have the menu just get taller and push the page down or up every time an item was selcted. Thoughts? |
Thank you so much for the reply. I really appreciate it. I agree, that having the menu get taller would be janky, but there was a version that worked well in F5. It wasn't called drilldown, but topbar had a function that would do something "similar". I have it working on a site now. http://www.woundhealingfoundation.org/ It just slides the whole menu over to the left, revealing the submenu. That way, height isn't an issue at all. |
There are a few options:
The Drilldown menu pattern probably isn't the best choice if any of those caveats don't work for you. I imagine Drilldown is best used with Off-canvas or in a layout where there's nothing underneath it. If you're using Drilldown in the middle of content, you're probably better served by a Dropdown or Accordion. |
Thanks @andycochran and @mitchartifextech for weighing in. I think for an off canvas example either 1 or 3 would work fine, since the background of the menu can be set to the background of the off canvas. For other use cases like when i menu is toggled open in page or just in page in general it still feels like finding the tallest child and setting a height feels the best. The jarring action of a slide that pushed the page content down still just feels super unappealing. |
Can we make it configurable, with the existing behavior being the default? |
We've gotten a number of requests for this now. We thought having the menu never change height would be more useful, but now it's looking like we need an option to configure that. If someone wants to implement a |
Great news! Thanks guys. I can do some beta testing if needed. I never felt it was too jarring and I come from a design/UI/UX background. Maybe if the menus are vastly different in height we leave it at default. If the menus are near the same height then toggle that new feature on :) 👍 |
data-auto-adjust-height and data-animate-height foundation#8607 foundation#7821
thx @designerno1 ;) @gakimball : Should we expect this pull request to be merged ? |
i think the feature will be in V6.3 Release |
Has this feature been added, if so how do we implement? |
This will be in v6.3, due out beginning of next month. You would implement by adding |
Only tried to work this out for a short time before I googled it..... phew cheers! |
@kball How do we use I tried to add Thanks! |
I've tried to set
My Drilldown menu doesn't show all of the items that the Dropdown menu does. To clarify: It looks like the height does change "automatically", but it is capped at 720.375px; If I remove the Is that a bug? |
Sorry, the issue I'm experiencing is not about the height but about the position of the Drilldown submenu. I opened issue #10478 |
How can we reproduce this bug?
What did you expect to happen?
I assumed the nested menu that animates in would grow tall enough to show all of the children.
What happened instead?
The root menu now has extra space below it, to accommodate the taller nested menu. For UI, this is not great, as it pushes content way below the menu. Ideally, the menu would grow if the nested section is taller.
Test case
http://lawyerb2b.atcdemo.com/ has the menu. Just shrink the screen below 1024px to show the Offcanvas menu.
See Screenshots here. http://screencast.com/t/4ap8MnzcfS http://screencast.com/t/8iAGj9C93mX
The text was updated successfully, but these errors were encountered: