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

Bug: cdk-overlay-container is broken when importing FundamentalNgxCoreModule in a module #11864

Open
LiorCapsiSAP opened this issue May 1, 2024 · 2 comments
Assignees

Comments

@LiorCapsiSAP
Copy link

Is this a bug, enhancement, or feature request?

BUG

Describe your proposal.

When opening a popover using a moduled component and importing FundamentalNgxCoreModule in component's module,
2 cdk-overlay-container divs are added to the DOM. At least one of them (some times 2) have a 'fd-message-toast-container' class on them. This causes different popovers to open in different containers (imagine a select control in a popup) so one hides the other.

Can you handle that on the application side

No.

Which versions of Angular and Fundamental Library for Angular are affected? Please, specify the exact version. (If this is a feature request, use current version.)

ng17.3 , fd 0.49.2

If this is a bug, please provide steps for reproducing it; the exact components you are using;

Open the popover's documentation in stackblitz (https://stackblitz.com/run?file=src%2Fapp%2Fpopover-programmatic-open-example.component.ts,src%2Findex.html).
add FundamentalNgxCoreModule to Imports.
open the popover.
Observe 2 divs with class 'cdk-overlay-container' . One of them with an additional 'fd-message-toast-container' class.
image

Please provide relevant source code (if applicable).

Please provide stackblitz example(s).

In case this is Accessibility related topic, did you consult with an accessibility expert? If not, please do so and share their recommendations.

Did you check the documentation and the API?

Did you search for similar issues?

Is there anything else we should know?

IMPORTANT: Please refrain from providing links or screenshots of SAP's internal information, as this project is open-source, and its contents are accessible to anyone.

@droshev
Copy link
Contributor

droshev commented May 23, 2024

@dpavlenishvili Can you check this issue?

@droshev droshev added this to the Sprint 134 - May 2024 milestone May 23, 2024
@LiorCapsiSAP
Copy link
Author

LiorCapsiSAP commented Jun 4, 2024

Adding to this issue- this happens when we import MessageToastModule which makes it totally unusable
image

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