From d3734c63fc18f1ad63b1b8765cdf03197be9e548 Mon Sep 17 00:00:00 2001
From: Jake Fink <jfink@bitwarden.com>
Date: Fri, 8 Oct 2021 08:49:15 -0400
Subject: [PATCH] Add SelectableLabel Custom Renderer to allow copy of note
 text (#1564)

* Add SelectableLabel Custom Renderer to allow copy of note text
- Remove SelectableLabelEffect

* Remove editor changes from text custom field
---
 src/Android/Android.csproj                    |  2 +-
 src/Android/Effects/SelectableLabelEffect.cs  | 23 -------
 .../Renderers/SelectableLabelRenderer.cs      | 25 +++++++
 src/App/Controls/SelectableLabel.cs           | 10 +++
 src/App/Effects/SelectableLabelEffect.cs      | 11 ---
 src/App/Pages/Vault/ViewPage.xaml             |  9 +--
 .../Renderers/SelectableLabelRenderer.cs      | 67 +++++++++++++++++++
 src/iOS.Core/iOS.Core.csproj                  |  1 +
 8 files changed, 106 insertions(+), 42 deletions(-)
 delete mode 100644 src/Android/Effects/SelectableLabelEffect.cs
 create mode 100644 src/Android/Renderers/SelectableLabelRenderer.cs
 create mode 100644 src/App/Controls/SelectableLabel.cs
 delete mode 100644 src/App/Effects/SelectableLabelEffect.cs
 create mode 100644 src/iOS.Core/Renderers/SelectableLabelRenderer.cs

diff --git a/src/Android/Android.csproj b/src/Android/Android.csproj
index 3ba99a4d4..da488db56 100644
--- a/src/Android/Android.csproj
+++ b/src/Android/Android.csproj
@@ -111,7 +111,6 @@
     <Compile Include="Autofill\SavedItem.cs" />
     <Compile Include="Effects\FabShadowEffect.cs" />
     <Compile Include="Effects\FixedSizeEffect.cs" />
-    <Compile Include="Effects\SelectableLabelEffect.cs" />
     <Compile Include="Effects\TabBarEffect.cs" />
     <Compile Include="Push\FirebaseMessagingService.cs" />
     <Compile Include="Receivers\ClearClipboardAlarmReceiver.cs" />
@@ -148,6 +147,7 @@
     <Compile Include="Utilities\AppCenterHelper.cs" />
     <Compile Include="Utilities\ThemeHelpers.cs" />
     <Compile Include="WebAuthCallbackActivity.cs" />
+    <Compile Include="Renderers\SelectableLabelRenderer.cs" />
   </ItemGroup>
   <ItemGroup>
     <AndroidAsset Include="Assets\FontAwesome.ttf" />
diff --git a/src/Android/Effects/SelectableLabelEffect.cs b/src/Android/Effects/SelectableLabelEffect.cs
deleted file mode 100644
index e14a39526..000000000
--- a/src/Android/Effects/SelectableLabelEffect.cs
+++ /dev/null
@@ -1,23 +0,0 @@
-using Android.Widget;
-using Bit.Droid.Effects;
-using Xamarin.Forms;
-using Xamarin.Forms.Platform.Android;
-
-[assembly: ExportEffect(typeof(SelectableLabelEffect), "SelectableLabelEffect")]
-namespace Bit.Droid.Effects
-{
-    public class SelectableLabelEffect : PlatformEffect
-    {
-        protected override void OnAttached()
-        {
-            if (Control is TextView textView)
-            {
-                textView.SetTextIsSelectable(true);
-            }
-        }
-
-        protected override void OnDetached()
-        {
-        }
-    }
-}
diff --git a/src/Android/Renderers/SelectableLabelRenderer.cs b/src/Android/Renderers/SelectableLabelRenderer.cs
new file mode 100644
index 000000000..ec4e8aa5e
--- /dev/null
+++ b/src/Android/Renderers/SelectableLabelRenderer.cs
@@ -0,0 +1,25 @@
+using System;
+using Android.Content;
+using Bit.App.Controls;
+using Bit.Droid.Renderers;
+using Xamarin.Forms;
+using Xamarin.Forms.Platform.Android;
+
+[assembly: ExportRenderer(typeof(SelectableLabel), typeof(SelectableLabelRenderer))]
+namespace Bit.Droid.Renderers
+{
+    public class SelectableLabelRenderer : LabelRenderer
+    {
+        public SelectableLabelRenderer(Context context) : base(context) { }
+
+        protected override void OnElementChanged(ElementChangedEventArgs<Label> e)
+        {
+            base.OnElementChanged(e);
+
+            if (Control != null)
+            {
+                Control.SetTextIsSelectable(true);
+            }
+        }
+    }
+}
diff --git a/src/App/Controls/SelectableLabel.cs b/src/App/Controls/SelectableLabel.cs
new file mode 100644
index 000000000..facf00b38
--- /dev/null
+++ b/src/App/Controls/SelectableLabel.cs
@@ -0,0 +1,10 @@
+using System;
+using Xamarin.Forms;
+
+namespace Bit.App.Controls
+{
+    public class SelectableLabel : Label
+    {
+
+    }
+}
diff --git a/src/App/Effects/SelectableLabelEffect.cs b/src/App/Effects/SelectableLabelEffect.cs
deleted file mode 100644
index 3f53ba706..000000000
--- a/src/App/Effects/SelectableLabelEffect.cs
+++ /dev/null
@@ -1,11 +0,0 @@
-using Xamarin.Forms;
-
-namespace Bit.App.Effects
-{
-    public class SelectableLabelEffect : RoutingEffect
-    {
-        public SelectableLabelEffect()
-            : base("Bitwarden.SelectableLabelEffect")
-        { }
-    }
-}
diff --git a/src/App/Pages/Vault/ViewPage.xaml b/src/App/Pages/Vault/ViewPage.xaml
index b04dd2108..54f58ff02 100644
--- a/src/App/Pages/Vault/ViewPage.xaml
+++ b/src/App/Pages/Vault/ViewPage.xaml
@@ -526,14 +526,9 @@
                                    StyleClass="box-header, box-header-platform" />
                         </StackLayout>
                         <StackLayout StyleClass="box-row">
-                            <Label
+                            <controls:SelectableLabel
                                 Text="{Binding Cipher.Notes, Mode=OneWay}"
-                                StyleClass="box-value"
-                                LineBreakMode="WordWrap">
-                                <Label.Effects>
-                                    <effects:SelectableLabelEffect />
-                                </Label.Effects>
-                            </Label>
+                                StyleClass="box-value"/>
                         </StackLayout>
                         <BoxView StyleClass="box-row-separator" />
                     </StackLayout>
diff --git a/src/iOS.Core/Renderers/SelectableLabelRenderer.cs b/src/iOS.Core/Renderers/SelectableLabelRenderer.cs
new file mode 100644
index 000000000..0813d0659
--- /dev/null
+++ b/src/iOS.Core/Renderers/SelectableLabelRenderer.cs
@@ -0,0 +1,67 @@
+using System;
+using System.ComponentModel;
+using Bit.App.Controls;
+using Bit.iOS.Core.Renderers;
+using UIKit;
+using Xamarin.Forms;
+using Xamarin.Forms.Platform.iOS;
+
+[assembly: ExportRenderer(typeof(SelectableLabel), typeof(SelectableLabelRenderer))]
+namespace Bit.iOS.Core.Renderers
+{
+    public class SelectableLabelRenderer : ViewRenderer<Label, UITextView>
+    {
+        UITextView uiTextView;
+
+        protected override void OnElementChanged(ElementChangedEventArgs<Label> e)
+        {
+            base.OnElementChanged(e);
+
+            if (Control == null)
+            {
+                uiTextView = new UITextView();
+            }
+
+            uiTextView.Selectable = true;
+            uiTextView.Editable = false;
+            uiTextView.ScrollEnabled = false;
+            uiTextView.TextContainerInset = UIEdgeInsets.Zero;
+            uiTextView.TextContainer.LineFragmentPadding = 0;
+            uiTextView.BackgroundColor = UIColor.Clear;
+
+            uiTextView.Text = Element.Text;
+            uiTextView.TextColor = Element.TextColor.ToUIColor();
+            switch (Element.FontAttributes)
+            {
+                case FontAttributes.None:
+                    uiTextView.Font = UIFont.SystemFontOfSize(new nfloat(Element.FontSize));
+                    break;
+                case FontAttributes.Bold:
+                    uiTextView.Font = UIFont.BoldSystemFontOfSize(new nfloat(Element.FontSize));
+                    break;
+                case FontAttributes.Italic:
+                    uiTextView.Font = UIFont.ItalicSystemFontOfSize(new nfloat(Element.FontSize));
+                    break;
+                default:
+                    uiTextView.Font = UIFont.BoldSystemFontOfSize(new nfloat(Element.FontSize));
+                    break;
+            }
+
+            SetNativeControl(uiTextView);
+        }
+
+        protected override void OnElementPropertyChanged(object sender, PropertyChangedEventArgs e)
+        {
+            base.OnElementPropertyChanged(sender, e);
+
+            if (e.PropertyName == Label.TextProperty.PropertyName)
+            {
+                if (Control != null && Element != null && !string.IsNullOrWhiteSpace(Element.Text))
+                {
+                    uiTextView.Text = Element.Text;
+                }
+            }
+        }
+
+    }
+}
diff --git a/src/iOS.Core/iOS.Core.csproj b/src/iOS.Core/iOS.Core.csproj
index 158109bdc..3fb702ac7 100644
--- a/src/iOS.Core/iOS.Core.csproj
+++ b/src/iOS.Core/iOS.Core.csproj
@@ -188,6 +188,7 @@
     <Compile Include="Views\StepperTableViewCell.cs" />
     <Compile Include="Views\SwitchTableViewCell.cs" />
     <Compile Include="Views\Toast.cs" />
+    <Compile Include="Renderers\SelectableLabelRenderer.cs" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\App\App.csproj">