chore(sync): merge upstream - #13
Open
Chloemlla wants to merge 3 commits into
Open
Conversation
…甲号切换并完善 macOS Liquid Glass 导航 (#701) * feat(macos): 使用原生 Liquid Glass 工具栏 * feat(macos): 展示完整原生导航 * fix(macos): 按平台设置主导航默认项 * fix(macos): 使用紧凑原生工具栏并移除重复上限变量 * fix(macos): 调整原生工具栏为舒适高度 * feat: 统一跨平台登录与 macOS 原生窗口 * refactor: 简化账户客户端与跨平台能力 * refactor: 收缩客户端初始化与宿主状态 * fix: 延后 mock 账户存储初始化 * fix: 稳定 mock 测试账户与 ViewModel 生命周期 * test: 对齐账户重构后的真实运行契约 * test: 在当前 Activity 验证重建后的状态栏 * test: 清理迁移后失效的状态注入 * test: 按动态标签页与解码后状态断言 * test: 精确定位屏蔽标签页交互 * feat: 支持保存和切换多个独立知乎账号 * feat: 修正连续朗读队列的 ktlint 格式 按问题顺序连续朗读并跳过已读回答的逻辑保持不变,仅收敛链式调用换行以通过 KtLint。 Constraint: 只修复本次 CI 失败点 Rejected: 一并整理其它无关改动 | 避免扩大提交面 Confidence: high Scope-risk: narrow Tested: ./gradlew :shared:runKtlintCheckOverCommonMainSourceSet; ./gradlew :shared:jvmTest --tests com.github.zly2006.zhihu.reading.ReadingPlayerTest; ./gradlew :app:compileLiteDebugKotlin * feat: 修正朗读队列链式调用换行 KtLint 在连续朗读队列的注册与拼接两处要求链式调用换行,按最新远端日志调整为统一风格,行为不变。 Constraint: 仅修复 CI 风格报错 Rejected: 保留单行链式调用 | 与当前 KtLint 配置不一致 Confidence: high Scope-risk: narrow Tested: ./gradlew :shared:runKtlintCheckOverCommonMainSourceSet * 防止消息图标被裁切并补上像素级回归 新增首页消息按钮的像素测试,实际截取屏幕并检查内容盒四角与内部像素,避免再次只靠尺寸/语义判断。顺手给内容盒补了独立 tag,便于稳定定位。\n\nConstraint: 需要真实截图证据来覆盖 badge/图标裁切\nRejected: 仅保留 semantics/size 检查 | 不能证明像素形状\nDirective: 只约束首页消息 action,不扩到无关 UI\nTested: ./gradlew :app:compileLiteDebugAndroidTestKotlin; ./gradlew :app:assembleLiteDebug :app:assembleLiteDebugAndroidTest; remote adb instrumentation on off AVD\nConfidence: high\nScope-risk: narrow * chore * fix: 修复合并冲突造成的功能回退 * fix: 恢复通知原生徽标渲染 * refactor: 删除过滤日志转发参数 * refactor: 删除消息提示的重复主线程转发 * refactor: 删除消息提示的主线程调度 * chore * feat: 将身份管理作为通用能力开放 * fix: 修复个人主页收起布局和窗口重开 * docs: 补充最终 UI 双态截图验收经验 * fix: 自定义个人主页折叠栏布局 * fix: 增加个人主页搜索区域渐变拦截层
* feat: 增加关闭窗口退出应用设置 * fix: 补充 macOS 关闭窗口退出设置搜索 * fix: 关闭 Android 状态栏对比度保护
|
Important Review skippedToo many files! This PR contains 482 files, which is 382 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. Usage-priced reviews support at most 300 files. ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (482)
You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated upstream sync by fork-sync.
zly2006masterChloemlla/zhihu-plus-plusmasterIf this PR has conflicts, please resolve manually. Clean PRs are auto-merged.