-
Notifications
You must be signed in to change notification settings - Fork 55
feat(FocusZone): Upgrade to the latest version #1772
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1772 +/- ##
=========================================
+ Coverage 69.77% 70.07% +0.3%
=========================================
Files 867 867
Lines 7354 7429 +75
Branches 2159 2161 +2
=========================================
+ Hits 5131 5206 +75
Misses 2215 2215
Partials 8 8
Continue to review full report at Codecov.
|
Please also update CHANGELOG |
@@ -26,6 +26,12 @@ This is a list of changes made to this Stardust copy of FocusZone in comparison | |||
- Handle keyDownCapture based on `shouldHandleKeyDownCapture` prop @sophieH29 ([#563](https://github.com/stardust-ui/react/pull/563)) | |||
- Add `bidirectionalDomOrder` direction allowing arrow keys navigation following DOM order @sophieH29 ([#1637](https://github.com/stardust-ui/react/pull/1647)) | |||
|
|||
### Upgrade `FocusZone` to the latest version from `fabric-ui` @sophieH29 ([#1772](https://github.com/stardust-ui/react/pull/1772)) | |||
- Restore focus on removing item ([OfficeDev/office-ui-fabric-react#7818](https://github.com/OfficeDev/office-ui-fabric-react/pull/7818)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any UTs added for 7818.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
### Upgrade `FocusZone` to the latest version from `fabric-ui` @sophieH29 ([#1772](https://github.com/stardust-ui/react/pull/1772)) | ||
- Restore focus on removing item ([OfficeDev/office-ui-fabric-react#7818](https://github.com/OfficeDev/office-ui-fabric-react/pull/7818)) | ||
- Reduce global event listeners ([OfficeDev/office-ui-fabric-react#7958](https://github.com/OfficeDev/office-ui-fabric-react/pull/7958)) | ||
- Track innerzones correctly ([OfficeDev/office-ui-fabric-react#8560](https://github.com/OfficeDev/office-ui-fabric-react/pull/8560)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No UTs from 8560.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't like an idea to add a public static method to FZ API to test it as here https://github.com/OfficeDev/office-ui-fabric-react/pull/8560/files#diff-2949b523d6a4d1f7e2e111abb6557158R82
If you're ok with that, I'll add it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would rather add it
Upgrade
FocusZone
to the latest version fromfabric-ui
library.Changes made based on these PRs: