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.

Sign up for GitHub

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

Jump to bottom

[待讨论] 原生图标相关适配问题 #18

Closed
Howard20181 opened this issue Feb 15, 2022 · 24 comments
Closed

[待讨论] 原生图标相关适配问题 #18

Howard20181 opened this issue Feb 15, 2022 · 24 comments
Labels
bug Something isn't working fixed The bug was fixed

Comments

@Howard20181
Copy link

原生图标是有颜色的，并不是完全黑的。MIUI的原生图标也是支持的
IMG_20220215_224927.jpg

可以禁用本模块，看看 https://github.com/lamprose/miultra 打开的MIUI原生图标
@fankes fankes added the wontfix This will not be worked on label Feb 15, 2022
@fankes
Copy link
Owner

fankes commented Feb 15, 2022

在 Android 12 中，官方的做法是将图标颜色的底色套在图标周围，如果你的版本不是 Android 12，可能会有问题，因为之前的版本我都是直接设置为纯色处理

@fankes fankes added the feature Future/Feature label Feb 15, 2022
@fankes
Copy link
Owner

fankes commented Feb 15, 2022

非常感谢，是我忽略了图标颜色的问题，这个问题将在稍后重新发布版本进行修复

@Howard20181
Copy link
Author

Android 11 MIUI 12.5

@fankes
Copy link
Owner

fankes commented Feb 15, 2022

该问题已修复

@fankes fankes closed this as completed Feb 15, 2022
@fankes fankes removed wontfix This will not be worked on feature Future/Feature labels Feb 15, 2022
@Howard20181
Copy link
Author

Howard20181 commented Feb 16, 2022 •
edited
Loading

感谢修复
但是还是发现一个不规范的影响不大的小问题，在低优先级可以折叠成一行的通知收起来是会变黑白的，展开才彩色，本模块不会有这个效果。如果不追求与原生完全一致这个可以忽略
效果如视频

Screenrecorder-2022-02-16-09-50-42-63.0.mp4

@fankes
Copy link
Owner

fankes commented Feb 16, 2022

感谢修复
但是还是发现一个不规范的影响不大的小问题，在低优先级可以折叠成一行的通知收起来是会变黑白的，展开才彩色，本模块不会有这个效果。如果不追求与原生完全一致这个可以忽略
效果如视频
https://user-images.githubusercontent.com/40033067/154181279-c8a75619-fa80-4832-8710-1ebfaffac04c.mp4

这个特性是 Android 12 之前的，之后完全修改了，不是很想适配之前的版本了，主要是感觉没必要了，通知展开折叠的情况也是需要判断的，MIUI 改的太乱了，如果你能接受就这样吧，有问题可以后期再说。

@fankes fankes added the feature Future/Feature label Feb 16, 2022
@Howard20181
Copy link
Author

Howard20181 commented Feb 16, 2022 •
edited
Loading

发现一个问题，折叠后通知刷新图标颜色会闪，看起来是黑白被设置成彩色

Screenrecorder-2022-02-16-21-40-29-365.0.mp4

@fankes
Copy link
Owner

fankes commented Feb 16, 2022

发现一个问题，折叠后通知刷新图标颜色会闪，看起来是黑白被设置成彩色

Screenrecorder-2022-02-16-21-40-29-365.0.mp4

裂开，我真的不想适配旧版本了，那就有时间修一下吧

@fankes fankes reopened this Feb 16, 2022
@fankes fankes added wontfix This will not be worked on bug Something isn't working To be discussed To be discussed issues labels Feb 16, 2022
@fankes
Copy link
Owner

fankes commented Feb 16, 2022

该问题被设置为待讨论状态，如果其他人有相同建议可在下方继续评论

@fankes fankes changed the title 原生图标是有颜色的 [待讨论] 原生图标相关适配问题 Feb 16, 2022
@fankes
Copy link
Owner

fankes commented Feb 26, 2022

使用相同的版本无法实现可折叠的效果，请问你使用的 Clash 版本是多少呢，最新版本已经没有折叠功能了，而且也没发现通知被折叠的可能性

@Howard20181
Copy link
Author

Howard20181 commented Feb 26, 2022 •
edited
Loading

使用相同的版本无法实现可折叠的效果，请问你使用的 Clash 版本是多少呢，最新版本已经没有折叠功能了，而且也没发现通知被折叠的可能性

低优先级的通知会被折叠，虽然MIUI上没有设置，但是这个流量通知好像默认就是低优先级的（应用在新建通知通道时可以指定重要程度）
设置为低优先级有个好处是可以隐藏图标（状态栏收起时）而不用完全关闭这个常驻通知

单个通知通道的重要程度设置在MIUI上被隐藏了，图示如下

Screenshot_2022-02-27-02-10-06-912_com.android.settings.jpg

可以使用这个Xposed模块加回设置入口
IMG_20220227_021352.jpg

@Howard20181
Copy link
Author

Howard20181 commented Feb 26, 2022 •
edited
Loading

不能上传apk只能打包一下了

Miui 12 Notification Importance_1.2.zip

作用域是设置com.android.settings

@naicfeng
Copy link
Contributor

复现条件比较苛刻，可作为低优先级TODO
首选建议迁移到 Android 12 ，折叠无问题

20220227025517.mp4

@naicfeng
Copy link
Contributor

naicfeng commented Feb 26, 2022 •
edited
Loading

不能上传apk只能打包一下了

Miui 12 Notification Importance_1.2.zip

作用域是设置com.android.settings

这个挺好……大家真的是为miui操碎了心……

已测试在miui13上也可以正常使用，最小化通知后默认不会折叠，但是图标会隐藏。不会闪烁，比较完美

@YifePlayte
Copy link

虽然兼容了原生图标颜色，但因为Material You的出现，个人更喜欢所有图标共用一套系统主题色。能否做到能开关通知图标颜色？

@Howard20181
Copy link
Author

复现条件比较苛刻，可作为低优先级TODO
首选建议迁移到 Android 12 ，折叠无问题

20220227025517.mp4

这个是原生样式的通知？
可惜小米9已经没有官方的Android 12了，除非抛弃MIUI否则用不上｡ﾟ(ﾟ´Д｀ﾟ)ﾟ｡

@Howard20181
Copy link
Author

虽然兼容了原生图标颜色，但因为Material You的出现，个人更喜欢所有图标共用一套系统主题色。能否做到能开关通知图标颜色？

原生是怎么处理的？我建议与原生的行为保持一致

@naicfeng
Copy link
Contributor

naicfeng commented Feb 27, 2022 •
edited
Loading

虽然兼容了原生图标颜色，但因为Material You的出现，个人更喜欢所有图标共用一套系统主题色。能否做到能开关通知图标颜色？

符合规范的APP可以自行决定图标是否上色，可以用测试工具测试一下。
如果做一个彩色开关，是否可作为降级解决图标闪色（掩耳盗铃一下？）

@naicfeng
Copy link
Contributor

虽然兼容了原生图标颜色，但因为Material You的出现，个人更喜欢所有图标共用一套系统主题色。能否做到能开关通知图标颜色？

原生是怎么处理的？我建议与原生的行为保持一致

这个就是miui13(a12)开原生通知的样式。
原生的行为是，如果应用没有给图标指定颜色(是透明)，就系统指定一个(图标内容是白，上色在背景)。图标指定自己为什么颜色背景就会上什么颜色。图标指定自己为彩色就是彩色

@Agoines
Copy link

Agoines commented Feb 27, 2022

发现一个问题，折叠后通知刷新图标颜色会闪，看起来是黑白被设置成彩色

Screenrecorder-2022-02-16-21-40-29-365.0.mp4

裂开，我真的不想适配旧版本了，那就有时间修一下吧

在 Android 12 也有类似问题，在通知被刷新的时候会闪烁一下。

@fankes
Copy link
Owner

fankes commented Feb 27, 2022

发现一个问题，折叠后通知刷新图标颜色会闪，看起来是黑白被设置成彩色

Screenrecorder-2022-02-16-21-40-29-365.0.mp4

裂开，我真的不想适配旧版本了，那就有时间修一下吧

在 Android 12 也有类似问题，在通知被刷新的时候会闪烁一下。

这个已经在说明里写明，是 MIUI 自己的问题，您不看说明的吗

@fankes
Copy link
Owner

fankes commented Feb 27, 2022

虽然兼容了原生图标颜色，但因为Material You的出现，个人更喜欢所有图标共用一套系统主题色。能否做到能开关通知图标颜色？

是可以做到，但是这不符合原生规范，破坏了应用自有通知图标颜色，就像 MIUI 的屎山一样，这不是这个模块要做的事情

@YifePlayte
Copy link

虽然兼容了原生图标颜色，但因为Material You的出现，个人更喜欢所有图标共用一套系统主题色。能否做到能开关通知图标颜色？

是可以做到，但是这不符合原生规范，破坏了应用自有通知图标颜色，就像 MIUI 的屎山一样，这不是这个模块要做的事情

好的，理解。

@fankes
Copy link
Owner

fankes commented Feb 28, 2022

该问题在最新版本已被修复，请核实并更新即可

@fankes fankes closed this as completed Feb 28, 2022
@fankes fankes removed wontfix This will not be worked on feature Future/Feature To be discussed To be discussed issues labels Feb 28, 2022
@fankes fankes added the fixed The bug was fixed label Mar 16, 2022
@fankes fankes mentioned this issue Mar 29, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working fixed The bug was fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@naicfeng @fankes @Howard20181 @Agoines @YifePlayte