xamarin/Xamarin.Forms

The order of 'IsEnabled' and 'Command' should not matter

Ouverte

#2 758 ouverte le 17 mai 2018

Voir sur GitHubGuide contributeur
 (24 commentaires) (18 réactions) (0 personne assignée)C# (1 926 forks)batch import
breakinge/5 :clock5:help wantedm/high impact :black_large_square:p/0t/bug :bug:up-for-grabs

Métriques du dépôt

Stars
 (5 644 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Description

I have two buttons:

<Button
    Text="Button1"
    Command="{Binding TheCommand}"
    IsEnabled="False" />

<Button
    Text="Button2"
    IsEnabled="False"
    Command="{Binding TheCommand}" />

For some reason, the IsEnabled property is overwritten in Button2 - this enables the button incorrectly.

Button1 is disabled this way, but when I trigger TheCommand.ChangeCanExecute(), the button enables.

Expected Behavior

Both buttons stay disabled.

The matrix of this should be an && operation:

TheCommand.CanExecute() IsEnabled Final State
True True Enabled
True False Disabled
False True Disabled
False False Disabled

Actual Behavior

The button with the command applied later overwrites the enabled state.

Basic Information

  • Version with issue: 2.5.x and 3.0
  • Last known good version: unsure
  • IDE: VS2017
  • Platform Target Frameworks: all

Guide contributeur

Direction de recherche
Le problème signale que l'ordre des propriétés 'IsEnabled' et 'Command' sur un Button affecte l'état d'activation final, ce qui ne devrait pas se produire. Le comportement attendu est une combinaison ET : le bouton est activé uniquement si IsEnabled local est true et Command.CanExecute() renvoie true. Pour corriger, examinez comment Button gère les changements de propriété pour IsEnabled et Command dans le code source de Xamarin.Forms (par exemple, Button.cs). La correction implique probablement de s'assurer que lorsque l'une des propriétés change, l'état d'activation final est recalculé comme l'ET logique des deux. Vérifiez s'il existe des solutions de contournement ou des commentaires existants dans le problème qui fournissent plus d'informations sur la base de code.
Stack technique
csharp
Domaine
mobile
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Xamarin.FormsC#MVVM
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30