Skip to content

Commit 4996314

Browse files
committed
remove unnecessary assertions
1 parent 6333a45 commit 4996314

5 files changed

Lines changed: 9 additions & 12 deletions

File tree

packages/cli/src/scripts/create-wrappers/PropTypesRenderer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ export class PropTypesRenderer extends AbstractRenderer {
2424
return {
2525
...s,
2626
name: 'children',
27-
} as CEM.Slot;
27+
};
2828
}
2929
return s;
3030
})

packages/main/src/components/AnalyticalTable/hooks/useDynamicColumnWidths.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@ const adjustColumnWidths = (instance: TableInstance) => {
531531
width,
532532
minWidth,
533533
maxWidth,
534-
} as AnalyticalTableColumnDefinition;
534+
};
535535
});
536536

537537
const hasData = data.length > 0;

packages/main/src/components/MessageItem/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ const MessageItem = forwardRef<ListItemCustomDomRef, MessageItemPropTypes>((prop
145145
>
146146
<FlexBox alignItems={FlexBoxAlignItems.Center} className={messageClasses}>
147147
<div className={classNames.iconContainer}>
148-
<Icon name={getIconNameForType(type as ValueState)} className={classNames.icon} mode={IconMode.Decorative} />
148+
<Icon name={getIconNameForType(type)} className={classNames.icon} mode={IconMode.Decorative} />
149149
</div>
150150
<FlexBox direction={FlexBoxDirection.Column} className={classNames.titleContainer}>
151151
{titleText && (

packages/main/src/components/ObjectPage/index.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -673,7 +673,7 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
673673
headerPinned || scrolledHeaderExpanded
674674
? `${topHeaderHeight + (headerCollapsed === true ? 0 : headerContentHeight) + tabContainerHeaderHeight}px`
675675
: `${topHeaderHeight + tabContainerHeaderHeight}px`,
676-
} as CSSProperties;
676+
};
677677
if (headerCollapsed === true && headerArea) {
678678
objectPageStyles[ObjectPageCssVariables.titleFontSize] = ThemingParameters.sapObjectHeader_Title_SnappedFontSize;
679679
}
@@ -693,7 +693,7 @@ const ObjectPage = forwardRef<ObjectPageDomRef, ObjectPagePropTypes>((props, ref
693693
const opNode = objectPageRef.current;
694694
if (!opNode) return;
695695

696-
if (e.relatedTarget && !e.currentTarget.contains(e.relatedTarget as Node)) {
696+
if (e.relatedTarget && !e.currentTarget.contains(e.relatedTarget)) {
697697
opNode.style.scrollPaddingBlock = '0px';
698698
// Fallback: Some (ui5-table) ui5wc components don't implement `relatedTarget` as expected.
699699
} else if (!e.relatedTarget) {

packages/main/src/components/SplitterLayout/index.tsx

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
import { debounce, useStylesheet, useSyncRef } from '@ui5/webcomponents-react-base';
44
import { clsx } from 'clsx';
5-
import type { CSSProperties } from 'react';
65
import { forwardRef, useEffect, useRef, useState } from 'react';
76
import { SplitterLayoutContext } from '../../internal/SplitterLayoutContext.js';
87
import { classNames, styleData } from './SplitterLayout.module.css.js';
@@ -89,12 +88,10 @@ const SplitterLayout = forwardRef<HTMLDivElement, SplitterLayoutPropTypes>((prop
8988
return (
9089
<SplitterLayoutContext.Provider value={{ vertical, reset }}>
9190
<div
92-
style={
93-
{
94-
flexDirection: vertical ? 'column' : 'row',
95-
...style,
96-
} as CSSProperties
97-
}
91+
style={{
92+
flexDirection: vertical ? 'column' : 'row',
93+
...style,
94+
}}
9895
title={title}
9996
{...rest}
10097
className={clsx(classNames.splitterLayout, className)}

0 commit comments

Comments
 (0)