CanExecute can not change Button clickability about maui HOT 9 CLOSED

@drasticactions Thanks! In my sample I did forget to set BindingContext, I didn't actually forget this in my actual project, the main problem was that I forgot to add [NotifyCanExecuteChangedFor(nameof(DoWorkCommand))].

from maui.

I'm glad you solved it! I'm also pleased that NotifyCanExecuteChangedFor exists; since my solution was kind of janky, that one makes more sense.

from maui.

Hi I'm an AI powered bot that finds similar issues based off the issue title.

Please view the issues below to see if they solve your problem, and if the issue describes your problem please consider closing this one and thumbs upping the other issue to help us prioritize it. Thank you!

Closed similar issues:

• Button Command CanExecute does not update the button visual on Windows (#6297), similarity score: 0.79
• Windows: Command ChangeCanExecute does not re-enable a disabled button (#7690), similarity score: 0.78
• Android-.NET6-Style issue when changing the button status (disabled->enabled) (#11336), similarity score: 0.76
• Setting 'CanExecute' doesn't change the status of Button controls when they are bound to MVVM Commands (#9753), similarity score: 0.76
• IsEnabled is unable to update visual appearance (#13971), similarity score: 0.75

Note: You can give me feedback by thumbs upping or thumbs downing this comment.

from maui.

Can repro this issue at Android platform on the latest 17.11.0 Preview 1.0(8.0.7/8.0.10/8.0.21).
BindingBreaksStyle.zip

from maui.

This looks related: #9753

from maui.

This looks related: #9753

Not quite the same, #9753 case is that the visual of the button has not changed, but in my case the button cannot be enabled/disabled based on the CanExecute change.

from maui.

FWIW I was able to get it working just fine

https://github.com/drasticactions/MauiRepros/tree/main/MvvmTestCanExecute

It may be that you didn't set up your model correctly? Maybe the library you are using is at fault? If you make a sample showing what you're doing that could help, or maybe run what I wrote above and see if it doesn't work for you?

from maui.

@drasticactions Hi, I made a sample, I use Mvvm Toolkit, and the [RelayCommand] feature.

public partial class MainViewModel : ObservableObject
{
    [ObservableProperty]
    private bool _canDoWork;  // binding to Switch button

    [RelayCommand(CanExecute = nameof(CanDoWork))]
    private static void DoWork()  // binding to 'Click me' button
    {
        Debug.WriteLine("DoWork");
    }
}

MauiIssue22652.zip

from maui.

Thank you for the sample! I believe you set up your view model incorrectly.

First, in your app page, your View Model isn't set as the BindingContext, it's null, so it will never work.

Then, in your ViewModel,

public partial class MainViewModel : ObservableObject
{
    [ObservableProperty]
    private bool _canDoWork;

    [RelayCommand(CanExecute = nameof(CanDoWork))]
    private static void DoWork()
    {
        Debug.WriteLine("DoWork");
    }
}

I am not well versed in the CommunityToolkit MVVM, but I believe you didn't set it up correctly. CanExecute gets called whenever someone calls for it to be raised, it's not done automatically. So when CanDoWork gets updated, you then need to raise the event on DoWork to be able to call CanExecute.

Changing it to this:

public partial class MainViewModel : ObservableObject
{
    [ObservableProperty]
    private bool _canDoWork;
    
    partial void OnCanDoWorkChanged(bool value)
    {
       this.DoWorkCommand.NotifyCanExecuteChanged();
    }

    [RelayCommand(CanExecute = nameof(CanDoWork))]
    private async void DoWork()
    {
        Debug.WriteLine("DoWork");
    }
}

Allows it to work. Likewise, DoWork can't be static, or else you can't call on the command for NotifyCanExecuteChanged. I would encourage you to read their documentation for more about it, but I do not believe this is a MAUI bug.

from maui.