Cleanup my mess

.vscode/tasks.json

@@ -5,13 +5,7 @@
       "label": "Typescript (oufr) watch",
       "type": "process",
       "command": "node",
-      "args": [
-        "./scripts/node_modules/typescript/bin/tsc",
-        "-p",
-        "packages/office-ui-fabric-react/tsconfig.json",
-        "-w",
-        "--noEmit"
-      ],
+      "args": ["./scripts/node_modules/typescript/bin/tsc", "-p", "packages/office-ui-fabric-react/tsconfig.json", "-w", "--noEmit"],
       "problemMatcher": "$tsc",
       "group": {
         "kind": "build",

common/changes/@uifabric/charting/charting_2018-05-22-23-27.json

@@ -8,4 +8,4 @@
   ],
   "packageName": "@uifabric/charting",
   "email": "kakje@microsoft.com"
-}
+}

common/changes/@uifabric/utilities/ftz-breakoutbug_2018-05-16-23-31.json

@@ -8,4 +8,4 @@
   ],
   "packageName": "@uifabric/utilities",
   "email": "benw@microsoft.com"
-}
+}

common/changes/office-ui-fabric-react/commandbar-fabricjs_2018-06-04-19-00.json → common/changes/office-ui-fabric-react/coachmarkSCSSFixes_2018-05-10-19-12.json

@@ -2,10 +2,10 @@
   "changes": [
     {
       "packageName": "office-ui-fabric-react",
-      "comment": "CommandBarPage: Remove fabric js reference.",
+      "comment": "Update Coachmark basic example and fix TeachingBubble SCSS selectors",
       "type": "patch"
     }
   ],
   "packageName": "office-ui-fabric-react",
-  "email": "lynam.emily@gmail.com"
-}
+  "email": "edwl@microsoft.com"
+}

common/changes/office-ui-fabric-react/applied-prettier_2018-06-02-22-17.json → common/changes/office-ui-fabric-react/demo2_2018-06-04-21-48.json

@@ -1,11 +1,11 @@
 {
   "changes": [
     {
+      "comment": "",
       "packageName": "office-ui-fabric-react",
-      "comment": "Added Prettier",
-      "type": "patch"
+      "type": "none"
     }
   ],
   "packageName": "office-ui-fabric-react",
-  "email": "mark@thedutchies.com"
-}
+  "email": "kchau@microsoft.com"
+}

common/changes/office-ui-fabric-react/ftz-breakoutbug_2018-05-16-23-31.json

@@ -2,11 +2,10 @@
   "changes": [
     {
       "packageName": "office-ui-fabric-react",
-      "comment":
-        "Fixed FocusTrapZone bug: If first child in the FTZ is a FocusZone and that FZ's last focused child is not the first focusable child, shift-tab would break out of the FTZ.",
+      "comment": "Fixed FocusTrapZone bug: If first child in the FTZ is a FocusZone and that FZ's last focused child is not the first focusable child, shift-tab would break out of the FTZ.",
       "type": "patch"
     }
   ],
   "packageName": "office-ui-fabric-react",
   "email": "benw@microsoft.com"
-}
+}

common/changes/office-ui-fabric-react/ftz-rememberlastfocused_2018-05-16-22-37.json

@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "office-ui-fabric-react",
+      "comment": "FocusTrapZone: Added new capability. When `FTZ.focus()` is called, it will pass focus to a descendant element.  The new prop `focusPreviouslyFocusedInnerElement` controls the descendant-choosing behavior.",
+      "type": "minor"
+    }
+  ],
+  "packageName": "office-ui-fabric-react",
+  "email": "benw@microsoft.com"
+}

common/changes/office-ui-fabric-react/jolore-contentBoxFix_2018-06-05-21-50.json

@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "office-ui-fabric-react",
+      "comment": "adding css fix to remove unnecessary scroll bars on calendar and datepicker components",
+      "type": "patch"
+    }
+  ],
+  "packageName": "office-ui-fabric-react",
+  "email": "jolore@microsoft.com"
+}

common/changes/office-ui-fabric-react/v-vibr-HoverCard-IEfix_2018-06-05-17-33.json

@@ -0,0 +1,11 @@
+{
+  "changes": [
+    {
+      "packageName": "office-ui-fabric-react",
+      "comment": "HoverCard: Removing unnecessary animation class causing a visual bug in IE browser.",
+      "type": "patch"
+    }
+  ],
+  "packageName": "office-ui-fabric-react",
+  "email": "v-vibr@microsoft.com"
+}

common/config/rush/version-policies.json

@@ -4,4 +4,4 @@
     "definitionName": "individualVersion",
     "lockedMajor": 6
   }
-]
+]

package.json

@@ -36,12 +36,13 @@
     "lint-staged": "^7.0.5"
   },
   "lint-staged": {
-    "*.{ts,tsx,js,json}": [
+    "*.{ts,tsx}": [
       "node ./scripts/lint-staged/prettier",
+      "node ./scripts/lint-staged/tslint",
       "git add"
     ],
-    "*.{ts,tsx}": [
-      "node ./scripts/lint-staged/tslint",
+    "*.{js,json}": [
+      "node ./scripts/lint-staged/prettier",
       "git add"
     ]
   }

packages/experiments/src/components/Nav/Nav.tsx

@@ -38,9 +38,9 @@ class NavComponent extends NavBase {
 
     return (
       <nav role="navigation">
-        {this.props.groups.map((group: ICustomNavLinkGroup, groupIndex: number) => {
+        { this.props.groups.map((group: ICustomNavLinkGroup, groupIndex: number) => {
           return this._renderGroup(group, groupIndex);
-        })}
+        }) }
       </nav>
     );
   }
@@ -108,21 +108,21 @@ class NavComponent extends NavBase {
 
     return (
       <NavLink
-        id={link.key}
-        content={linkText}
-        href={link.url}
-        target={link.target}
-        onClick={onClickHandler}
-        dataHint={dataHint}
-        dataValue={link.key}
-        ariaLabel={linkText}
-        {...ariaProps}
+        id={ link.key }
+        content={ linkText }
+        href={ link.url }
+        target={ link.target }
+        onClick={ onClickHandler }
+        dataHint={ dataHint }
+        dataValue={ link.key }
+        ariaLabel={ linkText }
+        { ...ariaProps }
         role="menu"
-        rootClassName={classNames.navItemRoot}
-        leftIconName={leftIconName}
-        rightIconName={rightIconName}
-        textClassName={classNames.navItemNameColumn}
-        iconClassName={classNames.navItemIconColumn}
+        rootClassName={ classNames.navItemRoot }
+        leftIconName={ leftIconName }
+        rightIconName={ rightIconName }
+        textClassName={ classNames.navItemNameColumn }
+        iconClassName={ classNames.navItemIconColumn }
       />
     );
   }
@@ -142,16 +142,16 @@ class NavComponent extends NavBase {
     link.disableAutoExpand = false;
 
     return (
-      <li role="listitem" key={link.key || linkIndex} title={linkText}>
-        {this._renderCompositeLink(link, linkIndex, nestingLevel)}
+      <li role="listitem" key={ link.key || linkIndex } title={ linkText }>
+        { this._renderCompositeLink(link, linkIndex, nestingLevel) }
         {// show child links
-        // 1. only for the first level and
-        // 2. if the link is expanded
-        nestingLevel == 0 && link.isExpanded ? (
-          <div className={AnimationClassNames.slideDownIn20}>
-            {this._renderLinks(link.links as INavLink[], ++nestingLevel)}
-          </div>
-        ) : null}
+          // 1. only for the first level and
+          // 2. if the link is expanded
+          nestingLevel == 0 && link.isExpanded ? (
+            <div className={ AnimationClassNames.slideDownIn20 }>
+              { this._renderLinks(link.links as INavLink[], ++nestingLevel) }
+            </div>
+          ) : null }
       </li>
     );
   }
@@ -165,7 +165,7 @@ class NavComponent extends NavBase {
 
     return (
       <ul role="list">
-        {links.map((link: INavLink, linkIndex: number) => {
+        { links.map((link: INavLink, linkIndex: number) => {
           if (enableCustomization && link.isHidden && !showMore) {
             // atleast one link is hidden
             this._hasAtleastOneHiddenLink = true;
@@ -178,7 +178,7 @@ class NavComponent extends NavBase {
           } else {
             return this._renderLink(link, linkIndex, nestingLevel);
           }
-        })}
+        }) }
       </ul>
     );
   }
@@ -205,15 +205,15 @@ class NavComponent extends NavBase {
     }
 
     return (
-      <div key={groupIndex}>
-        {isGroupHeaderVisible ? (
-          <div className={classNames.navGroupSeparatorRoot}>
-            <div className={classNames.navGroupSeparatorHrLine}>
-              {group.name ? <span className={classNames.navGroupSeparatorGroupName}>{group.name}</span> : null}
+      <div key={ groupIndex }>
+        { isGroupHeaderVisible ? (
+          <div className={ classNames.navGroupSeparatorRoot }>
+            <div className={ classNames.navGroupSeparatorHrLine }>
+              { group.name ? <span className={ classNames.navGroupSeparatorGroupName }>{ group.name }</span> : null }
             </div>
           </div>
-        ) : null}
-        {this._renderLinks(group.links, 0 /* nestingLevel */)}
+        ) : null }
+        { this._renderLinks(group.links, 0 /* nestingLevel */) }
       </div>
     );
   }

packages/experiments/src/components/Nav/NavBase.tsx

@@ -71,10 +71,8 @@ export class NavBase extends React.Component<INavProps, INavState> implements IN
       return false;
     }
 
-    return links.some(
-      (link: INavLink): boolean => {
-        return !link.isHidden;
-      }
-    );
+    return links.some((link: INavLink): boolean => {
+      return !link.isHidden;
+    });
   }
 }