Skip to content

feat: support toggling file list columns#12099

Open
A-Words wants to merge 4 commits into1Panel-dev:dev-v2from
A-Words:feat/12084-show-hide-file-column
Open

feat: support toggling file list columns#12099
A-Words wants to merge 4 commits into1Panel-dev:dev-v2from
A-Words:feat/12084-show-hide-file-column

Conversation

@A-Words
Copy link
Contributor

@A-Words A-Words commented Mar 6, 2026 •
edited
Loading

What this PR does / why we need it?

Summary of your change

  1. 为文件管理列表增加列显示/隐藏能力，支持切换 权限、大小、修改时间、备注 列
  2. 支持文件列表列配置的本地持久化，刷新页面后仍保留用户选择
image

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Mar 6, 2026

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Mar 6, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign zhengkunwang223 for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ssongliu
Copy link
Member

ssongliu commented Mar 6, 2026 •
edited
Loading

感谢贡献，这里建议和 容器 / 文件 列表一样，统一使用 fu-table-column-select 组件

@A-Words
Copy link
Contributor Author

A-Words commented Mar 6, 2026

感谢贡献，这里建议和 容器 / 文件 列表一样，统一使用 fu-table-column-select 组件

谢谢提醒，已改为使用 fu-table-column-select 组件

ssongliu
ssongliu reviewed Mar 6, 2026
View reviewed changes
frontend/src/views/host/file-management/index.vue Outdated
</el-table-column>
<el-table-column :label="$t('file.mode')" prop="mode" width="80">
<el-table-column
v-if="isColumnVisible('mode')"
Copy link
Member

@ssongliu ssongliu Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个判断是多余的吧

ssongliu
ssongliu reviewed Mar 6, 2026
View reviewed changes
frontend/src/views/host/file-management/index.vue Outdated
<el-table-column :label="$t('file.size')" prop="size" width="80" :sortable="'custom'">
<el-table-column
v-if="isColumnVisible('size')"
:label="$t('file.size')"
Copy link
Member

@ssongliu ssongliu Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多余判断

ssongliu
ssongliu reviewed Mar 6, 2026
View reviewed changes
frontend/src/views/host/file-management/index.vue Outdated

const pathRefs = ref<Record<string, any>>({});
const columns = ref<FileColumnConfig[]>([
{ label: i18n.global.t('file.mode'), value: 'mode', show: true },
Copy link
Member

@ssongliu ssongliu Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

可以不用赋值

@ssongliu
Copy link
Member

ssongliu commented Mar 6, 2026

感谢贡献，这里建议和 容器 / 文件 列表一样，统一使用 fu-table-column-select 组件

谢谢提醒，已改为使用 fu-table-column-select 组件

还有一些代码可以直接干掉，用 fix 限制一下某些不能隐藏的列
参考文档 https://fit2cloud-ui.github.io/fit2cloud-ui-plus/#/components/table

@A-Words
Copy link
Contributor Author

A-Words commented Mar 6, 2026

感谢贡献，这里建议和 容器 / 文件 列表一样，统一使用 fu-table-column-select 组件

谢谢提醒，已改为使用 fu-table-column-select 组件

还有一些代码可以直接干掉，用 fix 限制一下某些不能隐藏的列 参考文档 https://fit2cloud-ui.github.io/fit2cloud-ui-plus/#/components/table

已完成修改

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ssongliu ssongliu ssongliu left review comments

@wanghe-fit2cloud wanghe-fit2cloud Awaiting requested review from wanghe-fit2cloud

@zhengkunwang223 zhengkunwang223 Awaiting requested review from zhengkunwang223

Assignees

No one assigned

Labels

do-not-merge/release-note-label-needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@A-Words @ssongliu