Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: hide matched files/folders in sidebar #155

Merged
merged 2 commits into from
Mar 10, 2024
Merged

Conversation

liuycy
Copy link
Contributor

@liuycy liuycy commented Mar 3, 2024

refer to #146 (comment)

改动如下:

  • 目录树组件加了段逻辑,用来隐藏全局配置中正则匹配到的目录
  • 加了个 prop showHiddenFolder 用来区分 侧边栏移动/复制 操作里的目录树
  • 通过路径栏直接访问隐藏文件夹时,侧边栏会显示该文件夹(跳出后自动隐藏)

演示效果:

1.mp4

视频中使用的正则:

/\/README.md/i
/\/hide_folder_and_subfolder/i
/\/only_hide_parent_folder$/i

src/components/FolderTree.tsx Outdated Show resolved Hide resolved
@anwen-anyi
Copy link
Contributor

anwen-anyi commented Mar 4, 2024

有点儿怪,但是又双叒叕说不上那里怪 哈哈 😹

  • 不管是admin登录还是 游客 通过 路径进入 build 目录內

image


补充:似乎这次遇到的和这次修改没关系(有一半吧 哈哈)(使用Releases版本也会这样)

@liuycy liuycy marked this pull request as draft March 4, 2024 04:52
@pull-request-size pull-request-size bot added size/L and removed size/S labels Mar 4, 2024
@liuycy
Copy link
Contributor Author

liuycy commented Mar 4, 2024

有点儿怪,但是又双叒叕说不上那里怪 哈哈 😹

  • 不管是admin登录还是 游客 通过 路径进入 build 目录內

image

补充:似乎这次遇到的和这次修改没关系(有一半吧 哈哈)(使用Releases版本也会这样)

从路径可以跳进来,这个我怎么觉得是 feature 不是 bug 啊 😂,这样就相当于 隐身,留了个入口可以访问到这些文件了
如果真的要禁止访问的话,元信息的配置完全可以满足了吧

PS: 在正则后面加个 $ 可以只匹配到 build 结尾的路径,即 子目录 不会隐藏

@liuycy liuycy marked this pull request as ready for review March 4, 2024 07:56
@anwen-anyi
Copy link
Contributor

从路径可以跳进来,这个我怎么觉得是 feature 不是 bug 啊 😂,这样就相当于 隐身,留了个入口可以访问到这些文件了 如果真的要禁止访问的话,元信息的配置完全可以满足了吧

PS: 在正则后面加个 $ 可以只匹配到 build 结尾的路径,即 子目录 不会隐藏

对 是的 这个隐藏本身只是看不到它可以通过地址栏进来 不是Bug 设计就是这样的 🥇

对的如果真要隐藏是会使用元信息来隐藏的 😹

  • 这个隐藏只是有些文件夹在功能配合时候隐藏冗余展示时候挺有用的,例如添加了多个地址树然后使用别名将地址树文件夹都聚合到一个文件夹內把文件,不隐藏这些冗余的文件的话展开很多看着很乱 🤗
  • 你的提议在正则后加个$ 解决了另一半问题 哈哈 ❤️

现在木有问题了 两个都搞定了

@xhofe xhofe merged commit 6f7056e into AlistGo:main Mar 10, 2024
4 checks passed
@liuycy liuycy deleted the hotfix-146 branch March 12, 2024 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants