-
Notifications
You must be signed in to change notification settings - Fork 383
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
🚀 功能建议:会对低版本的Mac系统比如[10.X Catalina]做兼容吗? #251
Comments
Hello whoooami, Thank you for your first issue contribution 🎉 |
|
我看你勾选了提交 PR,如果能提交 PR,那我可以添加一个支持 macOS 10.x 系统版本的分支。 |
也好,不过我可能要花时间学习一下,时间周期会久一点 |
可以,如果你开发中遇到问题,随时可以向我们求助。 |
好的,瑞思拜 |
这些警告你只要按照系统的提示修复就可以,点击 Fix ,然后根据情况适配一下 macOS 10.15 示例: 2b92782 修复前修复后 |
好的,非常贴心的前置工作 |
同为catalina系统,正需要这个功能,什么时候能搞好? |
不确定,我这边没有 catalina 系统,也不方便调试,现在是 @whoooami 在着手做,如果大家能一起出力,提供一些帮助,应该能比较快完成。 |
图标也不一定非要自己设计,网上有一些开放的资源,例如阿里的 iconfont 先用 Xcode 跑起来,就成功了一大半。 |
好的,明白。我尽量把时间线提前 |
我先跑了一下本地环境
尝试重装xcode, 添加LD_VERIFY_BITCODE NO,依然无效,我可能需要专业开发帮我处理下环境问题 |
可能是第三方库 FIRAnalytics 版本太高了,使用了一些 Swift 库,而 macOS 10 不支持。 稍后我把第三方库的版本降一下试试。 |
好的,我们别的讨论方式吗? 我估计后续还会有一些问题要请教,还是都贴在这里就好? |
一般的技术问题可以直接这里讨论,方便查看和检索,如果涉及隐私,可以发邮件 [email protected],或者 Telegram tisfeng。 |
试了一下,发现 FIRAnalytics 无法在 macOS 10 使用,因此我直接移除了。 我更新了 macos-10.15 分支代码,解决了警告,剩下的你需要将调用到这两个图片方法的地方兼容一下。 + (NSImage *)ez_imageWithSymbolName:(NSString *)name {
CGSize size = CGSizeMake(EZAudioButtonImageWidth_16, EZAudioButtonImageWidth_16);
NSImage *image = [self ez_imageWithSymbolName:name size:size];
return image;
}
/// Return a system symbol image, if < macos(11.0), return a normal image with image name.
/// - Parameters:
/// - name: system symbol image name or assets image name
/// - size: image size
+ (NSImage *)ez_imageWithSymbolName:(NSString *)name size:(CGSize)size {
NSImage *image;
if (@available(macOS 11.0, *)) {
image = [NSImage imageWithSystemSymbolName:name accessibilityDescription:nil];
if (!CGSizeEqualToSize(size, CGSizeZero)) {
}
} else {
// Fallback on earlier versions
image = [NSImage imageNamed:name];
}
image = [image resizeToSize:size];
return image;
} |
好的,我更新了代码,编译还有其它报错。我再看下 |
遇到报错,可以先 Google 搜索一下,或者找 ChatGPT 聊一聊。 如果遇到难题无法解决,可以把问题贴出来,我来看看。 |
好的 |
你好,请问这个怎么样了? |
抱歉,还没有系统看代码。上次提到的问题我最近尽快debug一下,先出一版 |
ok,其实代码也不需要细看,主要是本地化字符串需要注意一下,你切到英文环境,各个功能都体验一遍就差不多了。 |
主要是上次运行起来主界面正常,我不太清楚是我不会使用还是功能性问题,所以暂时还没有更进一步了解 |
行,那就先这样,你可以把代码推上来,我们先发一个支持 macOS 10 的版本,如果后面有其他用户反馈问题,再来解决也不迟。没有就认为没问题,直接封存。 |
好的,等我晚点再跑一遍 |
macOS 10.15 这个分支可以运行在 macOS 10.15 系统,但可能有问题 #329 (comment) , 如果有能力改代码,请尝试自行修复。 这个分支后续我不会继续维护,只接受 PR。 |
请先确认以下事项
功能描述
🚀 功能建议:会对低版本的Mac系统比如[10.X Catalina]做兼容吗?
使用场景(可选)
No response
实现方案(可选)
No response
是否愿意提交 PR 实现该功能
The text was updated successfully, but these errors were encountered: