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.

Sign up for GitHub

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

Jump to bottom

Provide Top Layer containers #4950

Closed
wants to merge 2 commits into from
Closed

Provide Top Layer containers #4950

wants to merge 2 commits into from

Conversation

ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Dec 11, 2023 •
edited

☑️ Resolves

One of the ways to fix #4910

  • ⛔ Doesn't work with additional elements outside the container...
  • ⛔ Doesn't work with overflows

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🚧 Tasks

  • ...

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable

szaimen and others added 2 commits December 11, 2023 23:55
@susnux susnux added this to the 8.6.0 milestone Jan 24, 2024
@ShGKme
Copy link
Contributor Author

ShGKme commented Jan 25, 2024

Closing in flavor of an alternative #4953

@ShGKme ShGKme closed this Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
8.6.0
Development

Successfully merging this pull request may close these issues.

NcActions doesn't work inside an element with a focus trap in a general case
3 participants
@ShGKme @susnux @szaimen