-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
dropdown [appendToBody]="true" not showing #591
Comments
I would say |
Great, It's a requirement for my app. |
Hi @valorkin I need this, can you give advise how to implement this? I think you mention about new feature in angular2 to implement things like modals, popovers and this functionality that will change how to code it? Can you explain me in detail so I can work on this? Thanks! |
it'll use viewContainerRef as shown in the new service @valorkin's working on -> https://github.com/valor-software/ng2-bootstrap/blob/development/components/utils/components-helper.service.ts, which is the recommended way to do it now that DynamicComponentLoader has been deprecated. |
Hey, Do you know when it will be ready? |
Dear maintainers ;) Any ideas how do you want to implement it? It's very needed feature (actually totally required for me). I could help with that, do some PRs, but some general notes how you see it, would be nice. |
@valorkin @jtomaszewski Did you start working on a solution for this? I really need appendToBody for a project. Is there any way I can help? |
Yes, we did. If lucky it will be done till Monday |
@valorkin Any updates regarding this? ✨ |
yep, we updated a lot of code, added huge of test, written more docs |
@valorkin the dropdown options is indeed appending to the body but then but nothing happens when clicking on the dropdown, is that true ? |
as a status update: So I will be updateding dropdowns to work with it This week I want to finish with docs: current preview version |
@valorkin amazing, thanks |
Sorry for the annoying question, but are there any release plans to support |
Gabriel is looking into it |
I need this "dropdown-append-to-body" to work! Help! |
@umagon me too.. been waiting for a month, does not seem to be a priority for anyone. Angular 1 ui-bootstrap figured it out (https://angular-ui.github.io/bootstrap/#!#dropdown), waiting for whoever gets it done first for angular 2 and using that library. In the meantime I just added extra padding to my parent container if it has overflow hidden and a dropdown (as a workaround). |
@umagon @AlexViderman I understand it's frustrating but I went over the dd's code. In fact, there's quite a refactoring / code rewriting to make since @valorkin will use the new component loader on the dropdown menu and it wasn't designed to. I'm sure a PR will come soon. |
@ruffiem thanks. I'll keep waiting. my work-around is good enough for now. |
@AlexViderman I'll try to come up with something this week-end if @valorkin is too busy. |
Hey, it would be awesome. Just check how easy it could be in popover. |
@ruffiem , that would be a great feature, I am sure a lot of people are waiting on it as well. even if the implementation is not ideal, it would beat the DD hiding inside a div for now. @valorkin , totally understand, thank you for putting all the work into this library. as angular 2 grows and more people become to rely on your library for production sites, these little things become critical. |
@valorkin yes, the component loader looks very good 👍 I need to find a strategy to avoid regression on keyboard nav... Any idea ? |
Make it composeable |
`// todo: implement after porting position
with small class changes :) |
Testing and will roll-out new version of dropdown next week #1771 |
update please! When this will be available in stable release ? |
I will be wrapping up this today and publishing |
is this available for use ? |
If I switch to
[appendToBody]="true"
the dropdown menu is not rendered. Neither at the end of the body. Withfalse
it's rendering ok.The text was updated successfully, but these errors were encountered: