Fabric component: revert class change and make it backwards compatible

common/changes/@uifabric/utilities/fix-fabric-isfocusvisible_2018-04-25-01-59.json

@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "@uifabric/utilities",
+      "comment": "Adding `isDirectionalKeyCode` helper.",
+      "type": "minor"
+    }
+  ],
+  "packageName": "@uifabric/utilities",
+  "email": "dzearing@microsoft.com"
+}

common/changes/office-ui-fabric-react/fix-fabric-isfocusvisible_2018-04-25-01-59.json

@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "office-ui-fabric-react",
+      "comment": "Fabric: the isFocusVisible class is no added to the Fabric component again, to preserve backwards compatibility. Also fixing index file to export the types.",
+      "type": "minor"
+    }
+  ],
+  "packageName": "office-ui-fabric-react",
+  "email": "dzearing@microsoft.com"
+}

packages/office-ui-fabric-react/src/components/Fabric/Fabric.styles.ts

@@ -15,11 +15,13 @@ export const getStyles = (props: IFabricStyleProps): IFabricStyles => {
   const {
     theme,
     className,
+    isFocusVisible
   } = props;
 
   return {
     root: [
       'ms-Fabric',
+      isFocusVisible && 'is-focusVisible',
       theme.fonts.medium,
       {
         color: theme.palette.neutralPrimary,

packages/office-ui-fabric-react/src/components/Fabric/Fabric.test.tsx

@@ -0,0 +1,14 @@
+import * as React from 'react';
+import * as renderer from 'react-test-renderer';
+import { Fabric } from './Fabric';
+
+describe('Fabric', () => {
+  it('renders correctly', () => {
+    const component = renderer.create(
+      <Fabric>test</Fabric>
+    );
+    const tree = component.toJSON();
+    expect(tree).toMatchSnapshot();
+  });
+
+});

packages/office-ui-fabric-react/src/components/Fabric/Fabric.tsx

@@ -1,27 +1,63 @@
 import * as React from 'react';
 import {
   BaseComponent,
+  createRef,
   customizable,
   getNativeProps,
   divProperties,
-  classNamesFunction
+  classNamesFunction,
+  getWindow,
+  isDirectionalKeyCode
 } from '../../Utilities';
 import { getStyles } from './Fabric.styles';
 import { IFabricProps, IFabricStyleProps, IFabricStyles } from './Fabric.types';
 
 const getClassNames = classNamesFunction<IFabricStyleProps, IFabricStyles>();
 
 @customizable('Fabric', ['theme'])
-export class Fabric extends BaseComponent<IFabricProps, {}> {
+export class Fabric extends BaseComponent<IFabricProps, {
+  isFocusVisible: boolean;
+}> {
+  private _rootElement = createRef<HTMLDivElement>();
+
+  constructor(props: IFabricProps) {
+    super(props);
+    this.state = { isFocusVisible: false };
+  }
+
   public render() {
-    const classNames = getClassNames(getStyles, this.props as IFabricStyleProps);
+    const classNames = getClassNames(getStyles,
+      {
+        ...this.props as IFabricStyleProps,
+        ...this.state
+      });
     const divProps = getNativeProps(this.props, divProperties);
 
     return (
       <div
         { ...divProps }
         className={ classNames.root }
+        ref={ this._rootElement }
       />
     );
   }
+
+  public componentDidMount(): void {
+    const win = getWindow(this._rootElement.value);
+
+    if (win) {
+      this._events.on(win, 'mousedown', this._onMouseDown, true);
+      this._events.on(win, 'keydown', this._onKeyDown, true);
+    }
+  }
+
+  private _onMouseDown = (ev: MouseEvent): void => {
+    this.setState({ isFocusVisible: false });
+  }
+
+  private _onKeyDown = (ev: KeyboardEvent): void => {
+    if (isDirectionalKeyCode(ev.which)) {
+      this.setState({ isFocusVisible: true });
+    }
+  }
 }

packages/office-ui-fabric-react/src/components/Fabric/Fabric.types.ts

@@ -7,6 +7,7 @@ export interface IFabricProps extends React.HTMLAttributes<HTMLDivElement> {
 
 export interface IFabricStyleProps extends IFabricProps {
   theme: ITheme;
+  isFocusVisible: boolean;
 }
 
 export interface IFabricStyles {

packages/office-ui-fabric-react/src/components/Fabric/__snapshots__/Fabric.test.tsx.snap

@@ -0,0 +1,30 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Fabric renders correctly 1`] = `
+<div
+  className=
+      ms-Fabric
+      {
+        -moz-osx-font-smoothing: grayscale;
+        -webkit-font-smoothing: antialiased;
+        color: #333333;
+        font-family: 'Segoe UI', 'Segoe UI Web (West European)', 'Segoe UI', -apple-system, BlinkMacSystemFont, 'Roboto', 'Helvetica Neue', sans-serif;
+        font-size: 14px;
+        font-weight: 400;
+      }
+      & button {
+        font-family: inherit;
+      }
+      & input {
+        font-family: inherit;
+      }
+      & textarea {
+        font-family: inherit;
+      }
+      button {
+        overflow: visible;
+      }
+>
+  test
+</div>
+`;

packages/office-ui-fabric-react/src/components/Fabric/index.ts

@@ -1 +1,2 @@
 export * from './Fabric';
+export * from './Fabric.types';

packages/utilities/src/index.ts

@@ -31,6 +31,7 @@ export * from './hoist';
 export * from './hoistStatics';
 export * from './initializeFocusRects';
 export * from './initials';
+export * from './keyboard';
 export * from './language';
 export * from './math';
 export * from './memoize';
@@ -42,4 +43,4 @@ export * from './rtl';
 export * from './scroll';
 export * from './string';
 export * from './styled';
-export * from './warn';
+export * from './warn';

packages/utilities/src/initializeFocusRects.ts

@@ -1,19 +1,7 @@
 import { getWindow } from './dom';
-import { KeyCodes } from './KeyCodes';
-import { KeyboardEvent } from '../../../common/temp/node_modules/@types/react';
+import { isDirectionalKeyCode } from './keyboard';
 
 export const IsFocusVisibleClassName = 'ms-Fabric--isFocusVisible';
-const DirectionalKeyCodes = [
-  KeyCodes.up,
-  KeyCodes.down,
-  KeyCodes.left,
-  KeyCodes.right,
-  KeyCodes.home,
-  KeyCodes.end,
-  KeyCodes.tab,
-  KeyCodes.pageUp,
-  KeyCodes.pageDown
-];
 
 /**
  * Initializes the logic which:
@@ -51,14 +39,13 @@ function _onMouseDown(ev: MouseEvent): void {
   }
 }
 
-function _onKeyDown(ev: KeyboardEvent<Element>): void {
+function _onKeyDown(ev: KeyboardEvent): void {
   const win = getWindow(ev.target as Element);
 
   if (win) {
     const { classList } = win.document.body;
-    const isDirectionalKeyCode = DirectionalKeyCodes.indexOf(ev.which) > -1;
 
-    if (isDirectionalKeyCode && !classList.contains(IsFocusVisibleClassName)) {
+    if (isDirectionalKeyCode(ev.which) && !classList.contains(IsFocusVisibleClassName)) {
       classList.add(IsFocusVisibleClassName);
     }
   }

packages/utilities/src/keyboard.test.ts

@@ -0,0 +1,9 @@
+import { isDirectionalKeyCode } from './keyboard';
+import { KeyCodes } from './KeyCodes';
+
+describe('isDirectionalKeyCode', () => {
+  it('can return the expected value', () => {
+    isDirectionalKeyCode(KeyCodes.up);
+    isDirectionalKeyCode(KeyCodes.enter);
+  });
+});

packages/utilities/src/keyboard.ts

@@ -0,0 +1,20 @@
+import { KeyCodes } from './KeyCodes';
+
+const DirectionalKeyCodes: { [key: number]: number } = {
+  [KeyCodes.up]: 1,
+  [KeyCodes.down]: 1,
+  [KeyCodes.left]: 1,
+  [KeyCodes.right]: 1,
+  [KeyCodes.home]: 1,
+  [KeyCodes.end]: 1,
+  [KeyCodes.tab]: 1,
+  [KeyCodes.pageUp]: 1,
+  [KeyCodes.pageDown]: 1
+};
+
+/**
+ * Returns true if the keycode is a directional keyboard key.
+ */
+export function isDirectionalKeyCode(which: number): boolean {
+  return !!DirectionalKeyCodes[which];
+}