Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(merge-styles): export DeepPartialV2 from index of @fluentui/merge-styles #33806

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gyz0072004
Copy link
Member

@gyz0072004 gyz0072004 commented Feb 10, 2025

Previous Behavior

The DeepPartialV2 is used in typing of mergeStyles but not exported from index.

New Behavior

export type DeepPartialV2 from index.

Related Issue(s)

@gyz0072004 gyz0072004 requested review from dzearing and a team as code owners February 10, 2025 06:45
@gyz0072004 gyz0072004 force-pushed the master branch 2 times, most recently from 18dbf38 to 3edc2f5 Compare February 10, 2025 07:13
Copy link

github-actions bot commented Feb 10, 2025

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react
ExtendedPicker
96.915 kB
27.908 kB
96.915 kB
27.907 kB

-1 B
react
FloatingPicker
242.388 kB
69.002 kB
242.388 kB
69.003 kB

1 B
react
Fluent UI React (entire library)
1.019 MB
282.971 kB
1.019 MB
282.972 kB

1 B
react
SelectedItemsList
232.878 kB
67.891 kB
232.878 kB
67.893 kB

2 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react
ActivityItem
71.236 kB
23.357 kB
react
Announced
38.489 kB
13.296 kB
react
Autofill
15.459 kB
4.782 kB
react
Breadcrumb
202.318 kB
60.388 kB
react
Button
195.791 kB
56.627 kB
react
ButtonGrid
180.722 kB
54.627 kB
react
Calendar
121.875 kB
37.024 kB
react
Callout
84.323 kB
27.64 kB
react
Check
53.204 kB
17.848 kB
react
Checkbox
59.985 kB
19.903 kB
react
ChoiceGroup
65.494 kB
21.489 kB
react
ChoiceGroupOption
58.767 kB
19.362 kB
react
Coachmark
93.121 kB
29.414 kB
react
Color
7.789 kB
3.127 kB
react
ColorPicker
135.292 kB
42.21 kB
react
ComboBox
252.278 kB
72.315 kB
react
CommandBar
203.359 kB
60.141 kB
react
ContextualMenu
155.243 kB
48.202 kB
react
DatePicker
184.295 kB
56.15 kB
react
DateTimeUtilities
5.244 kB
1.849 kB
react
DetailsList
229.909 kB
65.781 kB
react
Dialog
211.717 kB
63.165 kB
react
Divider
19.603 kB
6.845 kB
react
DocumentCard
217.336 kB
64.44 kB
react
DragDrop
8.343 kB
2.724 kB
react
DraggableZone
34.303 kB
11.511 kB
react
Dropdown
234.695 kB
68.743 kB
react
Fabric
41.745 kB
14.366 kB
react
Facepile
210.893 kB
63.167 kB
react
FocusTrapZone
17.023 kB
5.911 kB
react
FocusZone
55.159 kB
17.492 kB
react
Grid
180.722 kB
54.627 kB
react
GroupedList
135.07 kB
40.7 kB
react
GroupedListV2
122.687 kB
37.788 kB
react
HoverCard
97.16 kB
30.787 kB
react
Icon
51.885 kB
17.272 kB
react
Icons
66.361 kB
24.397 kB
react
Image
46.904 kB
15.707 kB
react
Keytip
81.693 kB
26.766 kB
react
KeytipData
14.032 kB
4.582 kB
react
KeytipLayer
103.474 kB
32.015 kB
react
Keytips
106.238 kB
33.013 kB
react
Label
38.347 kB
13.257 kB
react
Layer
48.099 kB
16.367 kB
react
Link
39.682 kB
13.67 kB
react
List
39.371 kB
12.463 kB
react
MarqueeSelection
74.517 kB
22.433 kB
react
MessageBar
190.869 kB
57.103 kB
react
Modal
93.738 kB
30.271 kB
react
Nav
188.306 kB
56.485 kB
react
OverflowSet
33.385 kB
11.314 kB
react
Overlay
40.902 kB
14.095 kB
react
Panel
201.808 kB
60.106 kB
react
Persona
114.948 kB
36.532 kB
react
PersonaCoin
114.948 kB
36.532 kB
react
PersonaPresence
58.074 kB
19.384 kB
react
Pickers
299.085 kB
83.664 kB
react
Pivot
189.226 kB
57.267 kB
react
Popup
12.294 kB
4.195 kB
react
Positioning
22.807 kB
7.701 kB
react
PositioningContainer
73.827 kB
23.758 kB
react
ProgressIndicator
39.494 kB
13.55 kB
react
Rating
82.12 kB
26.127 kB
react
ResizeGroup
13.338 kB
4.377 kB
react
ResponsiveMode
8.13 kB
2.966 kB
react
ScrollablePane
55.557 kB
17.728 kB
react
SearchBox
189.118 kB
56.68 kB
react
SelectableOption
724 B
413 B
react
Selection
42.444 kB
12.278 kB
react
Separator
35.384 kB
12.146 kB
react
Shimmer
49.259 kB
16.268 kB
react
ShimmeredDetailsList
240.691 kB
68.532 kB
react
Slider
57.651 kB
19.217 kB
react
SpinButton
192.821 kB
57.775 kB
react
Spinner
41.775 kB
14.48 kB
react
Stack
42.074 kB
14.405 kB
react
Sticky
32.599 kB
10.504 kB
react
Styling
46.033 kB
15.135 kB
react
SwatchColorPicker
191.136 kB
58.151 kB
react
TeachingBubble
206.139 kB
61.063 kB
react
Text
36.908 kB
12.822 kB
react
TextField
80.79 kB
25.315 kB
react
Theme
43.493 kB
14.173 kB
react
ThemeGenerator
12.392 kB
4.126 kB
react
TimePicker
242.089 kB
70.072 kB
react
Toggle
46.225 kB
15.986 kB
react
Tooltip
87.437 kB
28.26 kB
react
Utilities
82.931 kB
25.148 kB
react
Viewport
23.888 kB
7.656 kB
react
WeeklyDayPicker
102.017 kB
31.834 kB
react
WindowProvider
1.059 kB
541 B
🤖 This report was generated against 0274540954f0bfade4d994c5a90d04dbc0936eff

Copy link

Pull request demo site: URL

@gyz0072004 gyz0072004 changed the title Export DeepPartialV2 from index of @fluentui/merge-styles chore(merge-styles): export DeepPartialV2 from index of @fluentui/merge-styles Feb 10, 2025
@@ -7,7 +7,7 @@ export type { IKeyframes } from './IKeyframes';
export type { IStyleFunction, IStyleFunctionOrObject } from './IStyleFunction';

// eslint-disable-next-line @typescript-eslint/no-deprecated
export type { DeepPartial } from './DeepPartial';
export type { DeepPartial, DeepPartialV2 } from './DeepPartial';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hotell WDYT? As I recall you added this type.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hotell Could you please help comment on this? Thanks.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@spmonahan @Hotell Kindly ping. Is there any further thought on the item?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: The DeepPartialV2 type is used in mergeStyles but not exported from index
2 participants