Skip to content

Commit 3b350c7

Browse files
abereghiciAlexandru BereghiciTheSisb
authored
chore: removed unnecessary type assertions (#3522)
Co-authored-by: Alexandru Bereghici <[email protected]> Co-authored-by: TheSisb <[email protected]>
1 parent 92c16ec commit 3b350c7

File tree

5 files changed

+19
-8
lines changed

5 files changed

+19
-8
lines changed

.changeset/eleven-ducks-retire.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
---
2+
"@twilio-paste/breadcrumb": patch
3+
"@twilio-paste/checkbox": patch
4+
"@twilio-paste/menu": patch
5+
"@twilio-paste/grid": patch
6+
"@twilio-paste/core": patch
7+
---
8+
9+
chore: removed unnecessary type assertions

packages/paste-core/components/breadcrumb/src/Breadcrumb.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,9 @@ const Breadcrumb = React.forwardRef<HTMLDivElement, BreadcrumbProps>(
125125
const [childrenCount, validChildren] = React.useMemo(
126126
() => [
127127
React.Children.count(children),
128-
React.Children.toArray(children).filter((child) => React.isValidElement(child) || typeof child === "string"),
128+
React.Children.toArray(children).filter(
129+
(child): child is React.ReactElement => React.isValidElement(child) || typeof child === "string",
130+
),
129131
],
130132
[children],
131133
);
@@ -135,7 +137,7 @@ const Breadcrumb = React.forwardRef<HTMLDivElement, BreadcrumbProps>(
135137
<Box aria-label="breadcrumb" {...safelySpreadBoxProps(props)} as="nav" element={element} ref={ref}>
136138
<Box alignItems="center" as="ol" display="inline-flex" listStyleType="none" margin="space0" padding="space0">
137139
{validChildren.map((child, index) => {
138-
return React.cloneElement(child as React.ReactElement<any>, {
140+
return React.cloneElement(child, {
139141
last: childrenCount === index + 1,
140142
key: keySeed(`breadcrumb-${index}`),
141143
parentElement: element,

packages/paste-core/components/checkbox/src/CheckboxGroup.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,8 @@ const CheckboxGroup = React.forwardRef<HTMLFieldSetElement, CheckboxGroupProps>(
9292
i18nRequiredLabel={i18nRequiredLabel}
9393
>
9494
{React.Children.map(children, (child, index) => {
95-
return React.isValidElement(child)
96-
? React.cloneElement(child as React.ReactElement<CheckboxProps>, {
95+
return React.isValidElement<CheckboxProps>(child)
96+
? React.cloneElement(child, {
9797
isSelectAll: isSelectAll && index === 0,
9898
isSelectAllChild: isSelectAll && orientation === "vertical" && index !== 0,
9999
})

packages/paste-core/components/menu/src/MenuGroup.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@ const MenuGroup = React.forwardRef<HTMLDivElement, MenuGroupProps>(
2121
ref={ref}
2222
>
2323
<Box display="flex" alignItems="center" paddingX="space70" paddingY="space30" cursor="default">
24-
{React.isValidElement(icon) ? (
24+
{React.isValidElement<GenericIconProps>(icon) ? (
2525
<Box flexShrink={0} size="sizeIcon30">
26-
{React.cloneElement(icon as React.ReactElement<GenericIconProps>, { color: "colorTextIcon" })}
26+
{React.cloneElement(icon, { color: "colorTextIcon" })}
2727
</Box>
2828
) : null}
2929
<Text

packages/paste-core/layout/grid/src/Grid.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@ const Grid = React.forwardRef<HTMLDivElement, GridProps>(
4545
const GridColumns = React.useMemo(
4646
() =>
4747
React.Children.map(children, (child) =>
48-
React.isValidElement(child)
49-
? React.cloneElement(child as React.ReactElement<ColumnProps>, {
48+
React.isValidElement<ColumnProps>(child)
49+
? React.cloneElement(child, {
5050
count: React.Children.count(children),
5151
gutter,
5252
vertical,

0 commit comments

Comments
 (0)