From cfcff98bb81963f55d7cdffe24f42b402e61d4d6 Mon Sep 17 00:00:00 2001
From: Jason Gore <jagore@microsoft.com>
Date: Thu, 9 Aug 2018 16:36:20 -0700
Subject: [PATCH] ColorPicker: Limit alpha decimal places (#5883)

* Limit decimal points displayed in ColorPicker.

* Change file.

* Revert multiplier change.

* Finish reversion.

* Switch to precision to fix screener test.

* Revert snapshots.
---
 ...g-5848-colorpicker-alpha-fix_2018-08-09-22-19.json | 11 +++++++++++
 .../src/components/ColorPicker/ColorPicker.base.tsx   |  4 ++--
 2 files changed, 13 insertions(+), 2 deletions(-)
 create mode 100644 common/changes/office-ui-fabric-react/jg-5848-colorpicker-alpha-fix_2018-08-09-22-19.json

diff --git a/common/changes/office-ui-fabric-react/jg-5848-colorpicker-alpha-fix_2018-08-09-22-19.json b/common/changes/office-ui-fabric-react/jg-5848-colorpicker-alpha-fix_2018-08-09-22-19.json
new file mode 100644
index 0000000000000..f3e88e61ea419
--- /dev/null
+++ b/common/changes/office-ui-fabric-react/jg-5848-colorpicker-alpha-fix_2018-08-09-22-19.json
@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "office-ui-fabric-react",
+      "comment": "ColorPicker: Limit decimal points shown for alpha.",
+      "type": "patch"
+    }
+  ],
+  "packageName": "office-ui-fabric-react",
+  "email": "jagore@microsoft.com"
+}
\ No newline at end of file
diff --git a/packages/office-ui-fabric-react/src/components/ColorPicker/ColorPicker.base.tsx b/packages/office-ui-fabric-react/src/components/ColorPicker/ColorPicker.base.tsx
index 89952d0e63404..8cdd3b19e365c 100644
--- a/packages/office-ui-fabric-react/src/components/ColorPicker/ColorPicker.base.tsx
+++ b/packages/office-ui-fabric-react/src/components/ColorPicker/ColorPicker.base.tsx
@@ -138,7 +138,7 @@ export class ColorPickerBase extends BaseComponent<IColorPickerProps, IColorPick
                     <TextField
                       className={classNames.input}
                       onBlur={this._onRGBAChanged}
-                      value={String(color.a)}
+                      value={String(color.a ? color.a.toPrecision(3) : color.a)}
                       componentRef={this._aText}
                       spellCheck={false}
                       ariaLabel={this.props.alphaLabel}
@@ -181,7 +181,7 @@ export class ColorPickerBase extends BaseComponent<IColorPickerProps, IColorPick
         r: Number(this._rText.current.value),
         g: Number(this._gText.current.value),
         b: Number(this._bText.current.value),
-        a: Number((this._aText && this._aText.current.value) || 100)
+        a: Number(this._aText.current.value || 100)
       })
     );
   };