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

Alert UI component doesn't support reactive bindings #2326

Open
Al12rs opened this issue Dec 2, 2024 · 0 comments
Open

Alert UI component doesn't support reactive bindings #2326

Al12rs opened this issue Dec 2, 2024 · 0 comments
Labels
Bug Something isn't working Dev UI

Comments

@Al12rs
Copy link
Contributor

Al12rs commented Dec 2, 2024

Bug Report

Summary

Our Alert.xaml.cs component doesn't support reactive bindings for Title and Body properties.
These can only be set using normal or compiled bindings from the view.

Details

This is caused by OnApplyTemplate() hiding parts of the template depending on the state of the properties (e.g. with IsNullOrEmpty()).
This happens when the template is first applied, meaning that the bindings from the code behind have not been setup yet (they happen on WhenActivated).

What is the expected behaviour?

If a binding is setup for the contents of the Title or the Body, Alert should set those sections of the template to visible upon set.

@Al12rs Al12rs added Bug Something isn't working Dev UI labels Dec 2, 2024
@Al12rs Al12rs added this to MVP Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Dev UI
Projects
Status: No status
Development

No branches or pull requests

1 participant