Skip to content

Handler Attached/Detached events #1200

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
Merged
rmarinho merged 1 commit into from
Jun 8, 2021
Merged

Handler Attached/Detached events #1200

rmarinho merged 1 commit into from
Jun 8, 2021

Conversation

@PureWeen
Copy link
Member

@PureWeen PureWeen commented Jun 8, 2021 •
edited
Loading

Description of Change

Implements #1198

Additions made

  • Overrides: VisualElement.OnHandlerAttached/OnHandlerDetached/OnHandlerAttaching/OnHandlerDetaching
  • Events: VisualElement.HandlerAttached/HandlerDetached/HandlerAttaching/HandlerDetaching
Event Description Use Cases
AttachingHandler Fires before handler has created native view This is used to make sure all properties on your xplat view have been initialized so they are correctly set on inception of the native view
AttachedHandler Fires after handler has created and set the native view The NativeView is present and all of the VirtualView properties have been applied. This is where you can interact with native element
DetachingHandler Fires before the handler is going to get removed from the virtual view The NativeView is about to get removed from this Virtual element so you should unwire any native events
DetachedHandler Fires after native handler has removed the native view

@rmarinho rmarinho merged commit 983387c into main Jun 8, 2021
@rmarinho rmarinho deleted the handler_lifecycle branch June 8, 2021 10:31
lytico pushed a commit to lytico/maui that referenced this pull request Jun 8, 2021
@PureWeen @lytico
Handler attach/detach events (dotnet#1200)
bf30a1e
@davidbritch davidbritch mentioned this pull request Jun 18, 2021
Closed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 26, 2023
@samhouts samhouts added the fixed-in-6.0.100-preview.5 Look for this fix in 6.0.100-preview.5! label Aug 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@rmarinho rmarinho rmarinho approved these changes

+1 more reviewer

@jsuarezruiz jsuarezruiz jsuarezruiz approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

fixed-in-6.0.100-preview.5 Look for this fix in 6.0.100-preview.5! platform/android

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@PureWeen @rmarinho @jsuarezruiz @samhouts