Skip to content

Commit aa80ed1

Browse files
committed
fix: update function calls to use optional chaining and improve CSS class handling
1 parent 91bd6ab commit aa80ed1

File tree

4 files changed

+3
-7
lines changed

4 files changed

+3
-7
lines changed

src/components/v2/appDetails/NodeTreeDetailTab.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ const NodeTreeDetailTab = ({
115115
return
116116
}
117117
if (stickyElementRef.current) {
118-
handleScrollToTop() // or your scroll container
118+
handleScrollToTop()
119119
}
120120
}
121121

src/components/v2/appDetails/k8Resource/nodeDetail/NodeDetail.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ const NodeDetailComponent = ({
369369

370370
const handleManifestEdit = () => {
371371
if (manifestFormConfigurationType === ConfigurationType.YAML) {
372-
handleStickDynamicTabsToTop()
372+
handleStickDynamicTabsToTop?.()
373373
}
374374
setManifestCodeEditorMode(ManifestCodeEditorMode.EDIT)
375375
}

src/components/v2/appDetails/k8Resource/nodeDetail/NodeDetailTabs/Manifest.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -767,7 +767,7 @@ const ManifestComponent = ({
767767
</div>
768768
) : (
769769
<div
770-
className="flexbox-col flex-grow-1"
770+
className={`flexbox-col flex-grow-1 ${isResourceBrowserView ? 'dc__overflow-auto' : ''}`}
771771
data-testid="app-manifest-container"
772772
style={{
773773
background: 'var(--terminal-bg)',

src/css/base.scss

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -354,10 +354,6 @@ a:focus {
354354
top: 36px;
355355
}
356356

357-
.dc__top-43 {
358-
top: 43px;
359-
}
360-
361357
.dc__top-44 {
362358
top: 44px;
363359
}

0 commit comments

Comments
 (0)