Skip to content

feat: 界面修改;国战调整;单挑补充;bug修复#3900

Open
mengxinzxz wants to merge 7 commits into
libnoname:dev/ui-adjustingfrom
mengxinzxz:main
Open

feat: 界面修改;国战调整;单挑补充;bug修复#3900
mengxinzxz wants to merge 7 commits into
libnoname:dev/ui-adjustingfrom
mengxinzxz:main

Conversation

@mengxinzxz
Copy link
Copy Markdown
Collaborator

@mengxinzxz mengxinzxz commented May 31, 2026

PR受影响的平台

重庆綦江山头、内蒙古高级初中、东北火锅鱼馆

诱因和背景

复活了,给之前的一些规划进行实现

PR描述

调整南华老仙【青书】的选择界面更方便点击查阅且与OL更为相似
image
image
调整胜负结算页面关于手牌查看的功能,取消以前只能往下翻看手牌和单机模式多数布局不能看自己剩余手牌的设定,改为在角色数据统计界面最右边添加手牌查看按钮,点击查看剩余手牌
image
image
image
优化get.poptip传入的name被解析后传入为innerHTML而非textContent形式
调整点将单挑模式若开启了单人控制则胜负结算默认游戏胜利
调整国战OL专属分包为“他山之石”,删除此分包中的限时且技能不变的武将
调整国战钟会【排异】【邀叛】和国战吴懿【转征】的技能效果为新版
调整神鲁肃【塌谟】不能换斗地主三号位(外服也一样不能换三号位不过描述没写出来)
删除无用APIget.groups,修复get.selectGroup的可选势力获取问题

PR测试

测了,能跑
(牢萌你Github账号怎么发紫啊😝)
(我说你PR提交不过我你信不信😡)

扩展适配

使用了get.groups的扩展须更改为其他方式

检查清单

  • commit中没有无用信息,和没有具体内容的“bugfix”
  • 我已经进行了充足的测试,且现有的测试都已通过
  • 若我拥有PR标签权限,则已确保为该PR打上标签;若我未拥有PR标签权限且该PR仍需继续提交内容,则已确保为该PR名称打上WIP直到本PR内容全部提交
  • 如果此次PR中添加了新的武将,则我已在character/rank.js中添加对应的武将强度评级,并对双人武将/复姓武将添加name:xxx的参数
  • 如果此次PR中添加了新的语音文件,则我已在lib.translate中加入语音文件的文字台词
  • 如果此次PR涉及到新功能的添加,我已在PR描述中写入详细文档
  • 如果此次PR需要扩展跟进,我已在扩展适配中写入详细文档
  • 如果这个PR解决了一个issue,我在诱因和背景中明确链接到该issue
  • 我保证该PR中没有随意修改换行符等内容,没有制造出大量的Diff
  • 我保证该PR遵循项目中.editorconfigeslint.config.mjsprettier.config.mjs所规定的代码样式,并且已经通过prettier格式化过代码

点将单挑单人控制时默认胜利,格式化single.js
调整对局结束界面的手牌查看功能:取消以前的翻到下面查阅的方式,改为显示在角色同行末尾点击显示
调整poptip传入字符改为修改innerHTML
将OL专属武将包更名为他山之石,删除此分包中的限时但技能不变的武将
调整国战钟会和国战吴懿的技能为新版
@rintim
Copy link
Copy Markdown
Member

rintim commented Jun 1, 2026

我不知道,但我想发1.11.4.1,所以给我进dev分支

image

@mengxinzxz mengxinzxz changed the title feat: 界面修改;国战调整;单挑补充 feat: 界面修改;国战调整;单挑补充;bug修复 Jun 1, 2026
Copy link
Copy Markdown
Member

@rintim rintim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

虽然但是,总归有点过于猛了,怕老戏直接合了,先请求更改再说(

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants