refactor: setting page
重构了设置页面的响应式布局,方便之后将部分设置项抽离至游戏页面
游戏页面的设置弹窗计划参考如下:
游戏页面添加了设置弹窗
目前,更改了设置项后,还不能直接应用到游戏中,怀疑之前的
store用法有问题,等排查后再合并
@orangelckc 感谢你的贡献,麻烦拉取最新代码,解决下现有冲突后提交,感谢!
@yaolifeng0629 已处理冲突,与当前主分支同步。
目前存在的问题:当前题目下,设置更改后,不能立即生效,但是进入下一题是会生效的。
可能需要核心团队重构当前store的使用方式后处理。
另外可以优化的一个点,建议将Modal全部更新为函数式动态挂载,目前会在页面中同时存在多个modal,可能会出现渲染层级问题
感谢!
-
这个 pr 需要拆分成2个功能 一个是重构 dialog 另外一个是添加 settings 弹窗
-
弹窗的背景色需要统一 现在 settings 的背景色和其他的弹窗背景色不一样 看起来很突兀
-
原有 settings page 可以先不用改动 后面需要重构 UI 因为现有的选择项越来越多 需要引入 tab 来分类
下一步建议:
先把这个 pr 改成只提交 重构 dialog 的功能
- 同步主分支并解决冲突(
git rebase main)✅
-
apps/client/components/DropMenu.vue(删掉了) -
apps/client/components/main/Tool.vue(恢复远端) -
apps/client/components/Navbar.vue(删除多余的信息,恢复远端)
- 强推远端代码(
git push -f orangelckc HEAD:kc/setting)✅ - 审查并补充 CommonModal 代码(适配深色模式)✅
- 重构 MessageBox 组件 ✅
- 移除多余文件 ✅
-
composables/messageBox/modal.ts -
components/main/MessageBox/useMessageBox.ts(函数式但只在测试文件中使用了) -
components/main/MessageBox/tests/message-box.spec.ts
- 更新文件位置 ✅
-
components/main/MessageBox/MessageBox.vue→components/main/MessageBox.vue
- 更新所有使用 CommonModal 的组件样式 ✅
- 移除新增的 setting 弹框代码 ✅