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

Messagebox object support #10212

Closed
BartaBlzs opened this issue Dec 30, 2024 · 4 comments · May be fixed by #10213
Closed

Messagebox object support #10212

BartaBlzs opened this issue Dec 30, 2024 · 4 comments · May be fixed by #10213
Labels
API suggestion Early API idea and discussion, it is NOT ready for implementation 🚧 work in progress Untriaged

Comments

@BartaBlzs
Copy link

Would be great if the MessageBox class had a method, that enables the developer to show not just strings, but any object.

@miloush miloush added the API suggestion Early API idea and discussion, it is NOT ready for implementation label Dec 30, 2024
@miloush
Copy link
Contributor

miloush commented Dec 30, 2024

I don't see any benefit to this, call .ToString() on the argument. For any API suggestions, please use the "API Suggestion" issue template.

@h3xds1nz
Copy link
Contributor

I agree with @miloush here, there's doesn't seem to be a visible benefit (but you may show one using an actual API suggestion).

Plus, it's a message box; it should serve for displaying messages, generally that's not a ToString override on an object.

@omariom
Copy link
Contributor

omariom commented Dec 30, 2024

It doesn't save much and it's confusing.
Can I pass a UIElement as the value? Will it apply a DataTemplate to the value?

@miloush
Copy link
Contributor

miloush commented Dec 30, 2024

@omarion you could and it would not, there is no reason why it should

@BartaBlzs BartaBlzs closed this as not planned Won't fix, can't repro, duplicate, stale Jan 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API suggestion Early API idea and discussion, it is NOT ready for implementation 🚧 work in progress Untriaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants