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

Add Scope handling for Portal Containers #829

Merged
merged 2 commits into from
Dec 21, 2023
Merged

Add Scope handling for Portal Containers #829

merged 2 commits into from
Dec 21, 2023

Conversation

metin-kale
Copy link
Collaborator

@metin-kale metin-kale commented Dec 18, 2023

In #781 we added portalling, but portals currently do not inherit any scope.

In this PR portal-containers get a scope of some defined parent tags.

We have two types of portals:

Bound to the ViewPort:

  • Modal
  • Toast

These Portals will inherit the Scope of the PortalRoot, which inherits the Scope of its parent (usually the main render Function).
Both the portalRoot() function and the global render() function got a scope parameter, so the user can set scopes as for every tag.

The second type of portals is bound to a reference element:

  • listbox
  • menu
  • popOver
  • tooltip

These portals inherit the scope of their reference element, to which they get aligned.

@Lysander Lysander added the headless All about headless components and foundations label Dec 18, 2023
@Lysander Lysander added this to the 1.0-RC13 milestone Dec 18, 2023
@metin-kale metin-kale requested a review from Lysander December 21, 2023 07:32
@metin-kale metin-kale marked this pull request as ready for review December 21, 2023 12:50
@metin-kale metin-kale merged commit 8cab86d into master Dec 21, 2023
2 checks passed
@Lysander Lysander changed the title scope handling for portal containers Add Scope handling for Portal Containers Jan 5, 2024
@Lysander Lysander deleted the portalscopes branch April 30, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
headless All about headless components and foundations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants