Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

添加 “在游戏列表中为自己100%完成的游戏添加背景色” 功能 #119

Merged
merged 12 commits into from
Jul 6, 2024

Conversation

kaikaiiiiiii
Copy link
Contributor

@kaikaiiiiiii kaikaiiiiiii commented Jun 27, 2024

在游列上添加个人游戏完成度的背景染色,效果如尾图:

代码中添加了两部分,

一部分是在个人页面更新游戏完成数据并添加到 GM_setValue 中的 savePersonalGameCompletions() 函数,该函数添加到了两处触发位置。

第二部分是在游戏列表中读取已保存数据并为对应游戏染底色。

其它行的变动为代码编辑器自行调整的缩进,我不知道怎么处理。// Update: lint used.

@LQV%AQY6HHEINR0I2M(6

@swsoyee
Copy link
Owner

swsoyee commented Jun 28, 2024

谢谢PR,我周末看一下

@swsoyee
Copy link
Owner

swsoyee commented Jul 1, 2024

idea非常好,大方向没什么问题,下面是一些建议的点

  1. 需要记得修改版本号。
  2. 如果使用背景色来区分的话,最好同时对夜间模式进行匹配。现在的话如果是夜间模式的时候背景右侧基本是白色的,看起来不清楚。
  3. 感觉没有说明的话,单纯添加一个颜色用户很可能搞不清楚这些颜色代表什么意思。比起颜色个人觉得是否添加一个【已完美】的标签会更好懂?

@kaikaiiiiiii
Copy link
Contributor Author

kaikaiiiiiii commented Jul 2, 2024

idea非常好,大方向没什么问题,下面是一些建议的点

  1. 需要记得修改版本号。
  2. 如果使用背景色来区分的话,最好同时对夜间模式进行匹配。现在的话如果是夜间模式的时候背景右侧基本是白色的,看起来不清楚。
  3. 感觉没有说明的话,单纯添加一个颜色用户很可能搞不清楚这些颜色代表什么意思。比起颜色个人觉得是否添加一个【已完美】的标签会更好懂?
  1. 我以为是 Owner 统一管理的。
  2. 我不知道还有夜间模式这东西,回头加个配色应该就行了。
  3. 这个我考虑过几种方案,当初考虑的最主要目标是在整个游列用滚轮上下翻动时,能『一目了然』地看清完成项。以此为目标:

方案一:在平台 BADGE 右边添加 xx%,mousetip =『游戏完成度』。实测发现,这种 badge 需要在上下翻扫时,集中目力去盯某个位置,感觉不符合原本的意图,放弃了。

方案二,在游戏图标上添加 XX% 的文字,或者白金杯图标等,一方面是 PS4 PS5 图标大小不一,另一方面遮挡图标也影响视觉,有些游戏图标还是很好看的,添加了显脏,放弃了。

方案三:在每项底部添加单独的几个像素的 Progress Bar,但会增加游列高度,同时每项都有个底部条还长短不一也很难看。如果只为 100% 的添加 bar,那和底色就没区别。

方案四:因为目标是『排除完成项』,所以也考虑过直接隐藏,但这样又要修改设置面板为用户提供开关比较麻烦,也没有这种底色显示已完成直观,就没做。

综上,我觉得目前的底色方案比较合理,真觉得说明不够的话,再加上方案一作为辅助也行。

PS: 关于颜色,我是参考了官方个人页面的 alert-success (无白金100%) 和 alert-info(白金100%),本来是直接给 tr 元素添加这两个类的,实测后发现效果不明显,就在原色基础上加深了些,又感觉整块底色很重,就面多加水加了渐变白。

@swsoyee
Copy link
Owner

swsoyee commented Jul 2, 2024

对于方案三也可以尝试把背景色的宽度做成动态宽度的,让背景色来当Progress bar就不用添加额外的像素了。同时在游戏标题左侧或者右侧加上方案一就很完美了。

@kaikaiiiiiii
Copy link
Contributor Author

kaikaiiiiiii commented Jul 5, 2024

更新夜间模式以及添加进度徽章。

chrome_3z5Satsmoc
chrome_aLpRKYRJeh

@kaikaiiiiiii
Copy link
Contributor Author

背景进度条效果好像确实也还行

chrome_lv5p3p3hDh
chrome_4ebPn00ZJa

Copy link
Owner

@swsoyee swsoyee left a comment

Choose a reason for hiding this comment

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

没问题,很好用!合并分支了。

@swsoyee swsoyee merged commit da1a788 into swsoyee:master Jul 6, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants