MaaAssistantArknights / maa-copilot-frontend

MAA Copilot Frontend
https://prts.plus
68 stars 50 forks source link

chore: update generated assets [skip ci] #255

Closed github-actions[bot] closed 6 months ago

github-actions[bot] commented 6 months ago

This is an auto-generated PR with updated static assets.

guansss commented 6 months ago

emmm,#252 把 json 格式化的参数去掉了,但好像提交之前又用 prettier 之类的格式化了一遍?

@Gemini2035 你是用的 prettier 吗

Gemini2035 commented 6 months ago

emmm,#252 把 json 格式化的参数去掉了,但好像提交之前又用 prettier 之类的格式化了一遍?

@Gemini2035 你是用的 prettier 吗

是指的生成后的operators.json吗 是的,使用了Prettier

guansss commented 6 months ago

我把这个 pr 里的 json 复制到本地试了一下,确实格式一样了,但干员顺序和之前有点不一样

image

看了一下,应该是这个地方

https://github.com/MaaAssistantArknights/maa-copilot-frontend/blob/9bf01b934a623ac561b52ba5befb700593ebd905/scripts/shared.ts#L84

Object.keys() 返回 json key 的顺序是不确定的,所以可能得拿到 key 之后排序一下

guansss commented 6 months ago

哦不对,后面有按拼音排序的,难道是拼音排序在不同平台上不一样?

当我没说,流星和流形的拼音是一样的,所以按拼音排序还不太够,还得额外加个 id 排序

guansss commented 6 months ago

修好力!#256

Gemini2035 commented 6 months ago

修好力!#256

有个不懂的点,这个干员顺序主要是什么作用啊。感觉顺序前后影响不大?

guansss commented 6 months ago

主要是为了在下拉框里展示,给用户看的,按拼音比较符合直觉

说起来现在快捷编辑里的列表也是按这个顺序,感觉可以加个按稀有度正/反排序的选项

Gemini2035 commented 6 months ago

主要是为了在下拉框里展示,给用户看的,按拼音比较符合直觉

说起来现在快捷编辑里的列表也是按这个顺序,感觉可以加个按稀有度正/反排序的选项

嗯,这样的话就干员信息还得从wiki上抓个稀有度字段下来。晚点试试