Skip to content

Commit 149889b

Browse files
committed
change getIconType function to just iconType
1 parent 1482270 commit 149889b

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

redisinsight/ui/src/components/navigation-menu/NavigationMenu.tsx

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ interface INavigations {
6262
dataTestId: string
6363
connectedInstanceId?: string
6464
onClick: () => void
65-
getIconType: () => IconType
65+
iconType: IconType
6666
onboard?: any
6767
featureFlag?: FeatureFlags
6868
}
@@ -124,7 +124,7 @@ const NavigationMenu = () => {
124124
onClick: () => handleGoPage(Pages.browser(connectedInstanceId)),
125125
dataTestId: 'browser-page-btn',
126126
connectedInstanceId,
127-
getIconType: () => BrowserIcon,
127+
iconType: BrowserIcon,
128128
onboard: ONBOARDING_FEATURES.BROWSER_PAGE,
129129
},
130130
{
@@ -135,7 +135,7 @@ const NavigationMenu = () => {
135135
dataTestId: 'workbench-page-btn',
136136
connectedInstanceId,
137137
isActivePage: activePage === `/${PageNames.workbench}`,
138-
getIconType: () => WorkbenchIcon,
138+
iconType: WorkbenchIcon,
139139
onboard: ONBOARDING_FEATURES.WORKBENCH_PAGE,
140140
},
141141
{
@@ -146,7 +146,7 @@ const NavigationMenu = () => {
146146
dataTestId: 'analytics-page-btn',
147147
connectedInstanceId,
148148
isActivePage: isAnalyticsPath(activePage),
149-
getIconType: () => SlowLogIcon,
149+
iconType: SlowLogIcon,
150150
featureFlag: FeatureFlags.envDependent,
151151
},
152152
{
@@ -157,7 +157,7 @@ const NavigationMenu = () => {
157157
dataTestId: 'pub-sub-page-btn',
158158
connectedInstanceId,
159159
isActivePage: activePage === pubSubPath,
160-
getIconType: () => PubSubIcon,
160+
iconType: PubSubIcon,
161161
onboard: ONBOARDING_FEATURES.PUB_SUB_PAGE,
162162
featureFlag: FeatureFlags.envDependent,
163163
},
@@ -171,7 +171,7 @@ const NavigationMenu = () => {
171171
onClick: () => handleGoPage(Pages.rdiStatistics(connectedRdiInstanceId)),
172172
dataTestId: 'pipeline-status-page-btn',
173173
isActivePage: activePage === `/${PageNames.rdiStatistics}`,
174-
getIconType: () => PipelineStatisticsIcon,
174+
iconType: PipelineStatisticsIcon,
175175
},
176176
{
177177
tooltipText: 'Pipeline Management',
@@ -181,7 +181,7 @@ const NavigationMenu = () => {
181181
handleGoPage(Pages.rdiPipelineManagement(connectedRdiInstanceId)),
182182
dataTestId: 'pipeline-management-page-btn',
183183
isActivePage: isPipelineManagementPath(),
184-
getIconType: () => PipelineManagementIcon,
184+
iconType: PipelineManagementIcon,
185185
},
186186
]
187187

@@ -193,7 +193,7 @@ const NavigationMenu = () => {
193193
onClick: () => handleGoPage(Pages.settings),
194194
dataTestId: 'settings-page-btn',
195195
isActivePage: activePage === Pages.settings,
196-
getIconType: () => SettingsIcon,
196+
iconType: SettingsIcon,
197197
featureFlag: FeatureFlags.envDependent,
198198
},
199199
]
@@ -217,7 +217,7 @@ const NavigationMenu = () => {
217217
tooltipProps={{ text: nav.tooltipText, placement: 'right' }}
218218
>
219219
<SideBarItemIcon
220-
icon={nav.getIconType()}
220+
icon={nav.iconType}
221221
aria-label={nav.ariaLabel}
222222
data-testid={nav.dataTestId}
223223
/>
@@ -261,7 +261,7 @@ const NavigationMenu = () => {
261261
className={styles.sideBarItem}
262262
>
263263
<SideBarItemIcon
264-
icon={nav.getIconType()}
264+
icon={nav.iconType}
265265
aria-label={nav.ariaLabel}
266266
data-testid={nav.dataTestId}
267267
/>

0 commit comments

Comments
 (0)