Skip to content

Commit 624249b

Browse files
JamesBurnsideJamesBurnside
authored
Update fluent and type-fest dependencies to latest (#4347)
1 parent a1afd43 commit 624249b

15 files changed

Lines changed: 1316 additions & 1223 deletions

File tree

‎change-beta/@azure-communication-react-33fdbed7-b090-4ef9-827d-5f1067a83789.json‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
{
2+
"type": "patch",
3+
"area": "improvement",
4+
"workstream": "",
5+
"comment": "Update fluent and type-fest dependencies to latest",
6+
"packageName": "@azure/communication-react",
7+
"email": "2684369+JamesBurnside@users.noreply.github.com",
8+
"dependentChangeType": "patch"
9+
}

‎change/@azure-communication-react-33fdbed7-b090-4ef9-827d-5f1067a83789.json‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
{
2+
"type": "patch",
3+
"area": "improvement",
4+
"workstream": "",
5+
"comment": "Update fluent and type-fest dependencies to latest",
6+
"packageName": "@azure/communication-react",
7+
"email": "2684369+JamesBurnside@users.noreply.github.com",
8+
"dependentChangeType": "patch"
9+
}

‎common/config/rush/pnpm-lock.yaml‎

Lines changed: 601 additions & 599 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎common/config/rush/variants/stable/pnpm-lock.yaml‎

Lines changed: 638 additions & 585 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/communication-react/package.json‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,13 @@
2626
"@azure/communication-common": "^2.3.1",
2727
"@azure/core-paging": "^1.5.0",
2828
"@azure/logger": "^1.0.4",
29-
"@fluentui/react-components": "^9.46.8",
30-
"@fluentui/react": "^8.115.2",
31-
"@fluentui/react-file-type-icons": "8.11.4",
32-
"@fluentui/react-hooks": "^8.6.36",
33-
"@fluentui/react-icons": "^2.0.231",
29+
"@fluentui/react-components": "^9.47.2",
30+
"@fluentui/react": "^8.117.0",
31+
"@fluentui/react-file-type-icons": "8.11.5",
32+
"@fluentui/react-hooks": "^8.7.0",
33+
"@fluentui/react-icons": "^2.0.233",
3434
"@fluentui/react-window-provider": "^2.2.18",
35-
"@fluentui-contrib/react-chat": "0.1.7",
35+
"@fluentui-contrib/react-chat": "0.1.9",
3636
"@griffel/react": "^1.5.20",
3737
"copy-to-clipboard": "^3.3.1",
3838
"dompurify": "^3.0.9",
@@ -156,7 +156,7 @@
156156
"ts-jest": "^29.1.2",
157157
"ts-node": "^10.9.2",
158158
"ts-patch": "^3.1.2",
159-
"type-fest": "^3.13.1",
159+
"type-fest": "^4.14.0",
160160
"typescript": "5.4.2",
161161
"typescript-transform-paths": "^3.4.7"
162162
},

‎packages/react-components/package.json‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,13 @@
3535
},
3636
"license": "MIT",
3737
"dependencies": {
38-
"@fluentui/react": "^8.115.2",
39-
"@fluentui/react-components": "^9.46.8",
40-
"@fluentui/react-file-type-icons": "8.11.4",
41-
"@fluentui/react-hooks": "^8.6.36",
42-
"@fluentui/react-icons": "^2.0.231",
38+
"@fluentui/react": "^8.117.0",
39+
"@fluentui/react-components": "^9.47.2",
40+
"@fluentui/react-file-type-icons": "8.11.5",
41+
"@fluentui/react-hooks": "^8.7.0",
42+
"@fluentui/react-icons": "^2.0.233",
4343
"@fluentui/react-window-provider": "^2.2.18",
44-
"@fluentui-contrib/react-chat": "0.1.7",
44+
"@fluentui-contrib/react-chat": "0.1.9",
4545
"@griffel/react": "^1.5.20",
4646
"@internal/acs-ui-common": "1.14.1-beta.0",
4747
"copy-to-clipboard": "^3.3.1",
@@ -126,7 +126,7 @@
126126
"source-map-explorer": "^2.5.0",
127127
"ts-jest": "^29.1.2",
128128
"ts-node": "^10.9.2",
129-
"type-fest": "^3.13.1",
129+
"type-fest": "^4.14.0",
130130
"typescript": "5.4.2",
131131
"webpack": "5.89.0"
132132
}

‎packages/react-components/src/components/VideoEffects/VideoBackgroundEffectsPicker.test.tsx‎

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,19 +45,25 @@ const TEST_PICKER_OPTIONS = [
4545

4646
describe('Background Picker should have the correct number of tiles', () => {
4747
test('Background Picker should have 7 tiles when items are set to wrap', async () => {
48-
const { container } = render(<_VideoBackgroundEffectsPicker options={TEST_PICKER_OPTIONS} itemsPerRow="wrap" />);
48+
const { container } = render(
49+
<_VideoBackgroundEffectsPicker options={TEST_PICKER_OPTIONS} itemsPerRow="wrap" ignoreFocusOnMount={true} />
50+
);
4951
expect(container.querySelectorAll('[data-ui-id="video-effects-item"]').length).toBe(TEST_PICKER_OPTIONS.length);
5052
});
5153

5254
test('Background Picker should have 7 tiles and 2 hidden tiles spanning 3 rows when items per row is set to 3', async () => {
53-
const { container } = render(<_VideoBackgroundEffectsPicker options={TEST_PICKER_OPTIONS} itemsPerRow={3} />);
55+
const { container } = render(
56+
<_VideoBackgroundEffectsPicker options={TEST_PICKER_OPTIONS} itemsPerRow={3} ignoreFocusOnMount={true} />
57+
);
5458
expect(container.querySelectorAll('[data-ui-id="video-effects-item"]').length).toBe(7);
5559
expect(container.querySelectorAll('[data-ui-id="video-effects-hidden-item"]').length).toBe(2);
5660
expect(container.querySelectorAll('[data-ui-id="video-effects-picker-row"]').length).toBe(3);
5761
});
5862

5963
test('Background Picker should have 7 tiles and 1 hidden tile spanning 4 rows when items per row is set to 2', async () => {
60-
const { container } = render(<_VideoBackgroundEffectsPicker options={TEST_PICKER_OPTIONS} itemsPerRow={2} />);
64+
const { container } = render(
65+
<_VideoBackgroundEffectsPicker options={TEST_PICKER_OPTIONS} itemsPerRow={2} ignoreFocusOnMount={true} />
66+
);
6167
expect(container.querySelectorAll('[data-ui-id="video-effects-item"]').length).toBe(7);
6268
expect(container.querySelectorAll('[data-ui-id="video-effects-hidden-item"]').length).toBe(1);
6369
expect(container.querySelectorAll('[data-ui-id="video-effects-picker-row"]').length).toBe(4);

‎packages/react-components/src/components/VideoEffects/VideoBackgroundEffectsPicker.tsx‎

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,15 @@ export interface _VideoBackgroundEffectsPickerProps {
5252
* Styles for the picker.
5353
*/
5454
styles?: _VideoBackgroundEffectsPickerStyles;
55+
56+
/**
57+
* Used only for jest tests. If `true`, the autofocus-on-mount will be ignored when the component mounts.
58+
* If not, jest tries to cleanup after the first render pass, but the component throws an error because
59+
* it tries to display a tooltip after that render pass and after the component is cleaned up.
60+
*
61+
* This can be removed once: https://github.com/microsoft/fluentui/issues/30896 is fixed.
62+
*/
63+
ignoreFocusOnMount?: boolean;
5564
}
5665

5766
/**
@@ -138,7 +147,12 @@ export const _VideoBackgroundEffectsPicker = (props: _VideoBackgroundEffectsPick
138147
{options.map((option, i) => {
139148
if (i === 0 && rowIndex === 0) {
140149
return (
141-
<_VideoEffectsItem {...option} itemKey={option.itemKey} key={option.itemKey} focusOnMount={true} />
150+
<_VideoEffectsItem
151+
{...option}
152+
itemKey={option.itemKey}
153+
key={option.itemKey}
154+
focusOnMount={true && !props.ignoreFocusOnMount}
155+
/>
142156
);
143157
}
144158
return <_VideoEffectsItem {...option} itemKey={option.itemKey} key={option.itemKey} />;

‎packages/react-composites/package.json‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,10 @@
5151
"@azure/core-paging": "^1.5.0",
5252
"@azure/core-rest-pipeline": "^1.13.0",
5353
"@azure/logger": "^1.0.4",
54-
"@fluentui/react-file-type-icons": "8.11.4",
55-
"@fluentui/react-hooks": "^8.6.36",
56-
"@fluentui/react-icons": "^2.0.231",
57-
"@fluentui/react": "^8.115.2",
54+
"@fluentui/react-file-type-icons": "8.11.5",
55+
"@fluentui/react-hooks": "^8.7.0",
56+
"@fluentui/react-icons": "^2.0.233",
57+
"@fluentui/react": "^8.117.0",
5858
"@internal/acs-ui-common": "1.14.1-beta.0",
5959
"@internal/calling-component-bindings": "1.14.1-beta.0",
6060
"@internal/calling-stateful-client": "1.14.1-beta.0",
@@ -152,7 +152,7 @@
152152
"ts-jest": "^29.1.2",
153153
"ts-loader": "^8.0.12",
154154
"ts-node": "^10.9.2",
155-
"type-fest": "^3.13.1",
155+
"type-fest": "^4.14.0",
156156
"typescript": "5.4.2",
157157
"webpack-cli": "^5.1.4",
158158
"webpack-dev-server": "4.15.1",

‎packages/storybook/package.json‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@
2626
"@azure/communication-common": "^2.3.1",
2727
"@azure/communication-identity": "^1.3.0",
2828
"@azure/communication-react": "1.14.1-beta.0",
29-
"@fluentui/react-components": "^9.46.8",
30-
"@fluentui/react-file-type-icons": "8.11.4",
31-
"@fluentui/react-hooks": "^8.6.36",
32-
"@fluentui/react-icons": "^2.0.231",
33-
"@fluentui/react": "^8.115.2",
29+
"@fluentui/react-components": "^9.47.2",
30+
"@fluentui/react-file-type-icons": "8.11.5",
31+
"@fluentui/react-hooks": "^8.7.0",
32+
"@fluentui/react-icons": "^2.0.233",
33+
"@fluentui/react": "^8.117.0",
3434
"@fluentui/theme-samples": "8.7.123",
3535
"@griffel/react": "^1.5.20",
3636
"@internal/fake-backends": "1.14.1-beta.0",

0 commit comments

Comments
 (0)
⚡