xamarin/Xamarin.Forms

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

Offen

#2.758 geöffnet am 17.05.2018

Auf GitHub ansehenContributor Guide
 (24 Kommentare) (18 Reaktionen) (0 zugewiesene Personen)C# (1.926 Forks)batch import
breakinge/5 :clock5:help wantedm/high impact :black_large_square:p/0t/bug :bug:up-for-grabs

Repository-Metriken

Stars
 (5.644 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem meldet, dass die Reihenfolge der Eigenschaften 'IsEnabled' und 'Command' auf einem Button den endgültigen Aktivierungszustand beeinflusst, was nicht passieren sollte. Das erwartete Verhalten ist eine UND Kombination: Der Button ist nur aktiviert, wenn sowohl das lokale IsEnabled true ist als auch Command.CanExecute() true zurückgibt. Zur Behebung untersuchen Sie, wie Button Eigenschaftsänderungen für IsEnabled und Command im Xamarin.Forms Quellcode (z. B. Button.cs) behandelt. Die Lösung besteht wahrscheinlich darin, sicherzustellen, dass bei Änderung einer der beiden Eigenschaften der endgültige Aktivierungszustand als logisches UND der beiden neu berechnet wird. Prüfen Sie, ob es im Problem vorhandene Workarounds oder Kommentare gibt, die weitere Einblicke in die Codebasis liefern.
Tech Stack
csharp
Domain
mobile
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Xamarin.FormsC#MVVM
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30