Skip to content

修复了自定义颜色面板打开时没有同步启动器主题颜色,间接修复了bug#5657#5777

Closed
pipixiaa22 wants to merge 1 commit intoHMCL-dev:mainfrom
pipixiaa22:fix/5756
Closed

修复了自定义颜色面板打开时没有同步启动器主题颜色,间接修复了bug#5657#5777
pipixiaa22 wants to merge 1 commit intoHMCL-dev:mainfrom
pipixiaa22:fix/5756

Conversation

@pipixiaa22
Copy link

修复了 bug#5657

JFXCustomColorPickerDialog 的 setCurrentColor 方法仅仅是设置了内部的 currentColorProperty 属性,但没有将这个颜色应用到颜色选择器 ( curvedColorPicker ) 的 UI 上。所以直接显式调用 curvedColorPicker.setColor(currentColor)即可。
这样下次打开的时候就是和启动同样的颜色,只有当自定义颜色和主题颜色不同的时候,关闭会进行修改

@Calboot
Copy link
Contributor

Calboot commented Mar 13, 2026

哥们 issue id 标错了吧

@pipixiaa22
Copy link
Author

是 5767 标错了

@Calboot
Copy link
Contributor

Calboot commented Mar 13, 2026

那重了啊

@WhatDamon
Copy link
Contributor

看你的实现是修复 #5766 , 且当前 PR 与我的 #5776 重复

@pipixiaa22
Copy link
Author

@WhatDamon 是的,以你的为准吧,这个修复解决了两个 bug

@WhatDamon
Copy link
Contributor

WhatDamon commented Mar 13, 2026

@WhatDamon 是的,以你的为准吧,这个修复解决了两个 bug

那么没有必要再开一个重复的 PR, 对于 #5767 的恰当修法是 #5775, 而 #5766 已经有 #5776, 而我在阅读了修改代码内容后发现实际上您的 PR 修复只处理了 #5766, 为了避免重复 PR 带来的混乱, 我建议您关闭此 PR

若您希望成为项目的贡献者, 可以找找 Issue 中的其他您能够解决问题, 在确定没有人针对您希望修复的 Issue 提交 PR 的前提下再进行贡献

@pipixiaa22
Copy link
Author

@WhatDamon 好的,我是第一次提交pr ,不懂这些,抱歉

@pipixiaa22 pipixiaa22 closed this Mar 13, 2026
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.

3 participants