Skip to content

全面应用圆角#5424

Draft
Calboot wants to merge 9 commits intoHMCL-dev:mainfrom
Calboot:island-ui
Draft

全面应用圆角#5424
Calboot wants to merge 9 commits intoHMCL-dev:mainfrom
Calboot:island-ui

Conversation

@Calboot
Copy link
Contributor

@Calboot Calboot commented Feb 4, 2026

image image image

@WhatDamon
Copy link
Contributor

有几个问题:

  • 如果本 PR 目的是推进 更新 UI 设计规范 #3683 那么必须强调当前 Island UI 设计是与目前呼声较高的 Material 3 设计其实相悖,此 PR 最好是作为一个可选外观而非一刀切
  • 滥用 Island UI 设计会导致软件中出现过多圆角,对视觉重心等有一定负面影响,若真要引入这样的设计,建议是将岛用于重要的地方,而不是每一个地方都来一个岛,设计可参考 IDEA 的解决方案
  • 圆角不协调,目前窗口圆角,组件圆角的圆角半径的不协调会导致视觉上很奇怪,即便不扣细节这并不明显
  • 目前来看冗余空间太多,特别体现在多于的嵌套,建议在 Draft 阶段在这方面进行一个评估,而非一味追求 “视觉上的好看”,因为效率也是人机交互中的重要一环
  • 当前 HMCL 设计是 Material 1、2、3 混合使用,导致阴影在此 PR 的设计中显得非常脏,尤其是 “陶瓦联机” 页面,在白色衬底下更加明显,虽然 Glavo 有意愿推进阴影去除,见 [Feature] 减少对阴影的使用 #5396
  • 此设计没有波及到 JFoenix 的部分 组件,虽然将 HMCL 与 JFoenix 强行耦合绝对不合适,但在 JFXColorPicker 等场景,应用此 PR 会产生较为严重的视觉的割裂
  • 图标上,如果目标是 “全面应用圆角”,可以考虑将所有 Material Symbols 替换成 Rounded 版本,即便我并不认为在这使用圆角是好的

以上内容供参考,但人机交互评估这一点,我认为若要推进 PR,必须细致考虑

@Calboot
Copy link
Contributor Author

Calboot commented Feb 4, 2026

有几个问题:

  • 如果本 PR 目的是推进 更新 UI 设计规范 #3683 那么必须强调当前 Island UI 设计是与目前呼声较高的 Material 3 设计其实相悖,此 PR 最好是作为一个可选外观而非一刀切
  • 滥用 Island UI 设计会导致软件中出现过多圆角,对视觉重心等有一定负面影响,若真要引入这样的设计,建议是将岛用于重要的地方,而不是每一个地方都来一个岛,设计可参考 IDEA 的解决方案
  • 圆角不协调,目前窗口圆角,组件圆角的圆角半径的不协调会导致视觉上很奇怪,即便不扣细节这并不明显
  • 目前来看冗余空间太多,特别体现在多于的嵌套,建议在 Draft 阶段在这方面进行一个评估,而非一味追求 “视觉上的好看”,因为效率也是人机交互中的重要一环
  • 当前 HMCL 设计是 Material 1、2、3 混合使用,导致阴影在此 PR 的设计中显得非常脏,尤其是 “陶瓦联机” 页面,在白色衬底下更加明显,虽然 Glavo 有意愿推进阴影去除,见 [Feature] 减少对阴影的使用 #5396
  • 此设计没有波及到 JFoenix 的部分 组件,虽然将 HMCL 与 JFoenix 强行耦合绝对不合适,但在 JFXColorPicker 等场景,应用此 PR 会产生较为严重的视觉的割裂
  • 图标上,如果目标是 “全面应用圆角”,可以考虑将所有 Material Symbols 替换成 Rounded 版本,即便我并不认为在这使用圆角是好的

以上内容供参考,但人机交互评估这一点,我认为若要推进 PR,必须细致考虑

  • 关于第一点,我觉得很难实现。我写这个 pr 本身就表明不是很喜欢 m3 的设计
  • 关于第二点和第四点,我可以考虑把主要界面的左右两个岛合并
  • 第三点和第六点我正在尝试解决

现在来看确实有很多需要慎重考虑的地方。这个 pr 的开发可能会暂时搁置,或分成小 pr,将我认为必要的部分逐渐应用进去

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants