Skip to content

Commit b990bb4

Browse files
Anush2303Anush
andauthored
fix(react-charts): remove redundant styles (#35476)
Co-authored-by: Anush <anushgupta@microsoft.com>
1 parent f0d9d1c commit b990bb4

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"type": "patch",
3+
"comment": "remove redundant styles",
4+
"packageName": "@fluentui/react-charts",
5+
"email": "anushgupta@microsoft.com",
6+
"dependentChangeType": "patch"
7+
}

packages/charts/react-charts/library/src/components/CommonComponents/useChartPopoverStyles.styles.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,6 @@ const useStyles = makeStyles({
3333
calloutContentRoot: {
3434
display: 'grid',
3535
overflow: 'hidden',
36-
// TODO: Confirm this style is needed. Shorthands are not used correctly, therefore this style has no effect.
37-
// ...shorthands.padding('11px 16px 10px 16px'),
3836
backgroundColor: tokens.colorNeutralBackground1,
3937
backgroundBlendMode: 'normal, luminosity',
4038
},

packages/charts/react-charts/library/src/components/Legends/useLegendsStyles.styles.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,6 @@ const useStyles = makeStyles({
4444
whiteSpace: 'nowrap',
4545
width: '100%',
4646
alignItems: 'center',
47-
// TODO: Confirm this style is needed. Shorthands are not used correctly, therefore this style has no effect.
48-
// ...shorthands.margin('-8px 0 0 -8px'),
4947
},
5048
legend: {
5149
// setting display to flex does not work

0 commit comments

Comments
 (0)