Skip to content

Commit

Permalink
[release-3.4] fix: fix the window have scroller when it have hpa card (
Browse files Browse the repository at this point in the history
…#4239)

* fix: fix the window have scroller when it have hpa card

Signed-off-by: harrisonliu5 <[email protected]>

* fix: fix horizontalpodautoscalers api version

Signed-off-by: harrisonliu5 <[email protected]>

---------

Signed-off-by: harrisonliu5 <[email protected]>
Co-authored-by: harrisonliu5 <[email protected]>
  • Loading branch information
ks-ci-bot and harrisonliu5 authored Oct 31, 2023
1 parent 1e832ea commit 879b287
Show file tree
Hide file tree
Showing 3 changed files with 40 additions and 15 deletions.
39 changes: 24 additions & 15 deletions src/pages/projects/components/Cards/HPA/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ import { Button, Icon, Dropdown, Menu, Notify } from '@kube-design/components'
import { Card } from 'components/Base'

import { getSuitableUnit, getValueByUnit } from 'utils/monitoring'
import { coreUnitTS } from 'utils'

import styles from './index.scss'

Expand Down Expand Up @@ -57,7 +58,8 @@ export default class HPACard extends React.Component {
getValue = (data, unitType) => {
const unit = getSuitableUnit(data, unitType)
const result = getValueByUnit(data, unit)
return unit ? `${result} ${unit}` : result
const unitText = coreUnitTS(result, unit)
return `${result} ${unitText}`
}

getHPAData = () => {
Expand Down Expand Up @@ -125,28 +127,35 @@ export default class HPACard extends React.Component {
handleCancel = () => {
const { detail, onDeleted } = this.props
this.store.delete(detail).then(() => {
Notify.success({ content: `${t('CANCELED_SUCCESSFULLY')}` })
Notify.success({ content: `${t('CANCEL_SUCCESSFUL')}` })
onDeleted()
})
}

renderOperations() {
const menus = this.getOperations()
const contenet = (
<Menu onClick={this.handleMoreClick}>
{menus.map(({ icon, text, show = true, ...rest }) => {
if (!show) return null
return (
<Menu.MenuItem key={text} {...rest}>
{icon && <Icon name={icon} type="light" />} {text}
</Menu.MenuItem>
)
})}
</Menu>
<div className={styles.menu}>
<Menu onClick={this.handleMoreClick}>
{menus.map(({ icon, text, show = true, ...rest }) => {
if (!show) return null
return (
<Menu.MenuItem key={text} {...rest}>
{icon && <Icon name={icon} type="light" />} {text}
</Menu.MenuItem>
)
})}
</Menu>
</div>
)

return (
<Dropdown theme="dark" content={contenet}>
<Dropdown
theme="dark"
content={contenet}
positionFixed
placement="bottomRight"
>
<Button type="ghost" icon="more" />
</Dropdown>
)
Expand Down Expand Up @@ -178,14 +187,14 @@ export default class HPACard extends React.Component {
}

render() {
const { className, loading } = this.props
const { className, loading, enableCancaleHPA = true } = this.props
const title = this.props.title || t('AUTOSCALING')

return (
<Card
className={classnames(styles.main, className)}
title={title}
operations={this.renderOperations()}
operations={enableCancaleHPA ? this.renderOperations() : null}
empty={t('NOT_ENABLE', { resource: t('AUTOSCALING') })}
loading={loading}
>
Expand Down
6 changes: 6 additions & 0 deletions src/pages/projects/components/Cards/HPA/index.scss
Original file line number Diff line number Diff line change
Expand Up @@ -38,3 +38,9 @@
line-height: 1.67;
}
}

.menu {
:global(.menu) {
min-width: 100px;
}
}
10 changes: 10 additions & 0 deletions src/utils/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -835,3 +835,13 @@ export const getDomTitle = name => {
}
return <span title={name}>{name}</span>
}

export const coreUnitTS = (value, unit) => {
let unitTxt = unit || ''

if (unit === 'core') {
unitTxt = value !== 1 ? t('CORE_PL') : t('CORE')
}

return unitTxt
}

0 comments on commit 879b287

Please sign in to comment.