hendrik/lens
1
0
Fork 0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00
Code Issues Projects Releases Wiki Activity

Fix ItemObjectList to not display column toggles for columns without IDs

Browse Source 
Signed-off-by: Sebastian Malton <sebastian@malton.name>
This commit is contained in:
Sebastian Malton 2022-10-03 10:31:16 -04:00
parent 3c0394374b
commit 78ebdd8405
1 changed files with 17 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
src/renderer/components/item-object-list/content.tsx
View File

@ -252,7 +252,7 @@ class NonInjectedItemListLayoutContent<
}
renderTableHeader() {
const { customizeTableRowProps, renderTableHeader, isSelectable, isConfigurable, store } = this.props;
const { customizeTableRowProps, renderTableHeader, isSelectable, isConfigurable, store, tableId } = this.props;
if (!renderTableHeader) {
return null;
@ -283,7 +283,10 @@ class NonInjectedItemListLayoutContent<
))
}
<TableCell className="menu">
{isConfigurable && this.renderColumnVisibilityMenu()}
{(isConfigurable && tableId)
? this.renderColumnVisibilityMenu(tableId)
: undefined
}
</TableCell>
</TableHead>
);
@ -341,8 +344,8 @@ class NonInjectedItemListLayoutContent<
return !isConfigurable || !tableId || !this.props.userStore.isTableColumnHidden(tableId, columnId, showWithColumn);
}
renderColumnVisibilityMenu() {
const { renderTableHeader = [], tableId } = this.props;
renderColumnVisibilityMenu(tableId: string) {
const { renderTableHeader = [] } = this.props;
return (
<MenuActions
@ -354,20 +357,16 @@ class NonInjectedItemListLayoutContent<
{
renderTableHeader
.filter(isDefined)
.map((cellProps, index) => (
!cellProps.showWithColumn && (
<MenuItem key={index} className="input">
<Checkbox
label={cellProps.title ?? `<${cellProps.className}>`}
value={this.showColumn(cellProps)}
onChange={(
tableId
? (() => cellProps.id && this.props.userStore.toggleTableColumnVisibility(tableId, cellProps.id))
: undefined
)}
/>
</MenuItem>
)
.filter((props): props is TableCellProps & { id: string } => !!props.id)
.filter(props => !props.showWithColumn)
.map((cellProps) => (
<MenuItem key={cellProps.id} className="input">
<Checkbox
label={cellProps.title ?? `<${cellProps.className}>`}
value={this.showColumn(cellProps)}
onChange={() => this.props.userStore.toggleTableColumnVisibility(tableId, cellProps.id)}
/>
</MenuItem>
))
}
</MenuActions>