-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: dark mode support #52
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 11 out of 26 changed files in this pull request and generated 2 comments.
Files not reviewed (15)
- app/toolbox/academic/index.tsx: Evaluated as low risk
- app/toolbox/academic/gpa.tsx: Evaluated as low risk
- app/+not-found.tsx: Evaluated as low risk
- app/(tabs)/_layout.tsx: Evaluated as low risk
- app/toolbox/academic/unified-exam.tsx: Evaluated as low risk
- app/(tabs)/qrcode.tsx: Evaluated as low risk
- app/(tabs)/toolbox.tsx: Evaluated as low risk
- app/(guest)/about.tsx: Evaluated as low risk
- app/toolbox/jiaxi-lecture.tsx: Evaluated as low risk
- app/devtools/push-tools.tsx: Evaluated as low risk
- app/toolbox/academic/grades.tsx: Evaluated as low risk
- app/toolbox/exam-room.tsx: Evaluated as low risk
- app/toolbox/empty-room.tsx: Evaluated as low risk
- app/toolbox/academic/credits.tsx: Evaluated as low risk
- app/devtools/index.tsx: Evaluated as low risk
突然想起来好像开屏页部分没有做深色模式? |
不用做强制切换,iOS 侧除了国产 app 基本都是跟随系统 |
这个 pr 待课表(#14)合入后再调整,目前已知存在问题的有
|
强制切换要做,可以后面再加。之前的自定义皮肤是基于深色模式的,可以参考一下 |
自定义皮肤感觉又要变成一个巨坑了……? |
本来就是,我反正不用自定义皮肤,但原来做的就不够好,但用户可以自适应,他们会选择比较容易看得清楚的壁纸 |
目前只是跟随系统,是否提供强制切换还有待商榷。
演示视频:
bandicam.2025-02-03.18-08-59-432.mp4
部分细节还需处理,等待组内美工介入。