菜单项 -> 所有站点设置 - 请指向 /admin/site_settings/category/all_results

只是一个小小的烦恼 :grimacing:

点击侧边栏导航菜单…
discourse-All-site-settings-menu-button

…时,我(已过滤)到达 /admin/site_settings/category/required 而不是 /admin/site_settings/category/all_results,这感觉不对。

…然后我就很难过

…在经历了这些难过时刻之后…

…我变得更加难过,因为我意识到我的短期记忆力已不如从前。

我知道这只是件小事,但摆脱这种反复出现的时刻(并节省那 :grimacing: 多余的点击)将有助于我变得更加

discourse-site-settings-all-feature-request

2 个赞

这确实很奇怪!@hugh 有什么原因吗,还是一个小疏忽?

1 个赞

我想它可能会记住你上次所在的标签页。 但事实并非如此。

我想知道那会不会不那么令人困惑,因为“all”只显示 100 个设置,而不是全部。

是的。一次性加载所有 4,168 条明显会更糟!… https://github.com/discourse/discourse/blob/main/config/site_settings.yml

不确定这是故意的还是无意的。通过一个草稿 PR https://github.com/discourse/discourse/pull/35263 很容易修复。但是的,我认为主要问题(我将留给 Hugh 处理)是这个问题:

另外,对于 @denvergeeks,您是否知道可以使用 Ctrl+/ 管理员搜索快捷方式搜索任何设置?

image

2 个赞

另一个想法:我们在德语翻译中删除了“所有”一词,因为侧边栏空间有限。也许对其他语言也这样做是有意义的。

1 个赞

在我看来,这似乎是一个疏忽,它确实应该出现在页面的“全部”部分。它从“必需”开始的一个不幸的副作用是,当您过滤结果时,它仍然是“必需”,因此您可能看不到您正在寻找的过滤条件。

@martin 在此处的 PR 修复了此问题,因此我乐于将其合并:FEATURE: Link to "All settings" for real in the admin sidebar by martin-brennan · Pull Request #35263 · discourse/discourse · GitHub

这是需要考虑的事情,但我想至少暂时保留“全部”。因为页面可以过滤以显示任何站点设置,所以“全部”仍然是一个正确的标签。不过,我很乐意根据更多反馈重新评估这一点。

3 个赞

好的,现在已经合并了!

3 个赞