remove Both option since it doesnt makes sense.
changes been discussed with as well
MODIFY
WalletWasabi.Fluent/Controls/QuestionControl.axaml.cs
WalletWasabi.Fluent/Controls/QuestionControl.axaml.cs
@@ -9,8 +9,7 @@ public enum HighlightedButton
{
None,
YesButton,
- NoButton,
- Both
+ NoButton
}
public class QuestionControl : ContentControl
@@ -59,4 +58,25 @@ public class QuestionControl : ContentControl
get => GetValue(HighlightButtonProperty);
set => SetValue(HighlightButtonProperty, value);
}
+
+ public QuestionControl()
+ {
+ UpdatePseudoClasses(HighlightButton);
+ }
+
+ protected override void OnPropertyChanged<T>(AvaloniaPropertyChangedEventArgs<T> change)
+ {
+ base.OnPropertyChanged(change);
+
+ if (change.Property == HighlightButtonProperty)
+ {
+ UpdatePseudoClasses(change.NewValue.GetValueOrDefault<HighlightedButton>());
+ }
+ }
+
+ private void UpdatePseudoClasses(HighlightedButton highlightedButton)
+ {
+ PseudoClasses.Set(":yes", highlightedButton == HighlightedButton.YesButton);
+ PseudoClasses.Set(":no", highlightedButton == HighlightedButton.NoButton);
+ }
}

MODIFY
WalletWasabi.Fluent/Views/Login/PasswordFinder/ContainsNumbersView.axaml
WalletWasabi.Fluent/Views/Login/PasswordFinder/ContainsNumbersView.axaml
<c:QuestionControl Content="Does your password contain numbers?"
YesCommand="{Binding YesCommand}"
NoCommand="{Binding NoCommand}"
- ImageIcon="{Binding Icon}"
- HighlightButton="Both" />
+ ImageIcon="{Binding Icon}" />
</c:ContentArea>
</UserControl>

MODIFY
WalletWasabi.Fluent/Views/Login/PasswordFinder/ContainsSymbolsView.axaml
WalletWasabi.Fluent/Views/Login/PasswordFinder/ContainsSymbolsView.axaml
<c:QuestionControl Content="Does your password contain special characters?"
YesCommand="{Binding YesCommand}"
NoCommand="{Binding NoCommand}"
- ImageIcon="{Binding Icon}"
- HighlightButton="Both" />
+ ImageIcon="{Binding Icon}" />
</c:ContentArea>
</UserControl>

remove Both option since it doesnt makes sense.

changes been discussed with @soosr as well