[Proposal] SelectAllTextEffect

SelectAllTextEffect

• Proposed
• Prototype
• Implementation
  • iOS Support
  • Android Support
  • macOS Support
  • Windows Support
• Unit Tests: Not Started
• Sample
• Documentation: Not Started

Summary

Highlights all text when an InputView is selected

Detailed Design

SelectAllTextEffect.shared.cs

public class SelectAllTextEffect : RoutingEffect
{
}

SelectAllTextEffect.android.cs

public class SelectAllTextEffect : PlatformEffect
{
  EditText EditText => (EditText)Control;
  
  protected override void OnAttached() => EditText?.SetSelectAllOnFocus(true);
  
  protected override void OnDetached() => EditText?.SetSelectAllOnFocus(false);
}

SelectAllTextEffect.ios.cs

public class SelectAllTextEffect : PlatformEffect
{
  protected override void OnAttached() => ApplyEffect(true);
  
  protected override void OnDetached() => ApplyEffect(false);
  
  void ApplyEffect(bool apply) => ApplyToControl(Control, apply);

  bool ApplyToControl<T>(T controlType, bool apply) => controlType switch
  {
	UITextField textField => ApplyToUITextField(textField, apply),
	UITextView => ApplyToUITextView(apply),
	_ => throw new NotSupportedException($"Control of type: {controlType?.GetType()?.Name} is not supported by this effect.")
  };
}

Usage Syntax

[VladislavAntonyuk]

@brminnick, @pictos seems like it can be closed as completed. any objections?

[pictos]

I’ll use this one to draw the API to implement our Effects as Behavior and after that go through all other Effects and see if it makes sense for all.

As we discussed on the last CommunityStandup