PCL2 icon indicating copy to clipboard operation
PCL2 copied to clipboard

新功能:简单的本地 Mod 筛选

Open tangge233 opened this issue 1 year ago • 14 comments

效果

image

还有一件事

Close #679

tangge233 avatar Aug 04 '24 15:08 tangge233

挺好 ~~,有人做了我想做的事~~。 看了眼 CI 构建的版本,提一点 UI 方面的建议:

  1. 我觉得可以不用调换功能按钮与搜索框的位置
  2. 如果全选时没有打开的卡片会给人一种类似 虚空选择 的感觉
  3. 可以默认显示全部,然后给一个下拉框实现切换 Filter?

CI 构建版本下载指北:直接下载 Release 462f985.zip 源地址 https://github.com/tangge233/PCL2/actions/runs/10237324912

MoYuan-CN avatar Aug 04 '24 15:08 MoYuan-CN

如果用下拉框切换的话界面可能有点难看.jpg

tangge233 avatar Aug 04 '24 16:08 tangge233

image 我想的是这样的……

因为 PR 的实现方法和我预想的完全不一样,所以先不检查了 orz

LTCatt avatar Aug 04 '24 17:08 LTCatt

好像需要首次加载过有缓存后,第二次才可以直接获取 Mod CanUpdate 情况 看来要改改(

tangge233 avatar Aug 05 '24 00:08 tangge233

image 解决了奇奇怪怪的问题,同时较符合了龙猫的预期 :) ~~界面我不太会写,龙猫帮我~~

tangge233 avatar Aug 05 '24 05:08 tangge233

The number of mods that can be updated should not be added to the number of all mods.

The current behavior is, for example, if I have three mods and one mod can be updated, then the total number of mods will become four.

Oh,that was my mistake.I will fix it later.

tangge233 avatar Aug 05 '24 06:08 tangge233

@LTCatt 弄好了,大概是你要的样子( Have a look: https://github.com/tangge233/PCL2/actions/runs/10247847302/artifacts/1776002365

tangge233 avatar Aug 05 '24 11:08 tangge233

I have no other opinions at the moment.我目前没有其他意见。

Similar to the previous example, there are three mods, one disabled and one updatable. When we select the disabled or updatable label, only one mod will be shown in the list, but Ctrl+A will prompt that three mods are selected.与前面的示例类似,有三个模组,一个已禁用,一个可更新。当我们选择禁用或可更新的标签时,列表中只会显示一个模组,但 Ctrl+A 会提示已选择三个模组。

From the commit history, this seems to be intentional, but I personally feel that the logic is a bit strange.从提交历史来看,这似乎是有意为之,但我个人觉得逻辑有点奇怪。

Fixed. I was a bit too careless.

tangge233 avatar Aug 05 '24 11:08 tangge233

LTCatt 弄好了,大概是你要的样子( Have a look: https://github.com/tangge233/PCL2/actions/runs/10247847302/artifacts/1776002365

不对啊,你这开源版本咋还显示快照版提示啊

SqrtTan avatar Aug 06 '24 12:08 SqrtTan

随机测试这么多次,应该不会再出问题了

tangge233 avatar Aug 06 '24 12:08 tangge233

LTCatt 弄好了,大概是你要的样子( Have a look: https://github.com/tangge233/PCL2/actions/runs/10247847302/artifacts/1776002365

不对啊,你这开源版本咋还显示快照版提示啊

默认就这么显示,如有修改意见,可以 PR 或 Issue

tangge233 avatar Aug 06 '24 12:08 tangge233

LTCatt 弄好了,大概是你要的样子( Have a look: https://github.com/tangge233/PCL2/actions/runs/10247847302/artifacts/1776002365

不对啊,你这开源版本咋还显示快照版提示啊

~你是没用过开源版吗,本来就是这样的~ #4181

确实没用过,我不会写C#

SqrtTan avatar Aug 07 '24 03:08 SqrtTan