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

fix: 调整用户级别显示 #1555

Merged
merged 2 commits into from
Sep 2, 2023
Merged

fix: 调整用户级别显示 #1555

merged 2 commits into from
Sep 2, 2023

Conversation

IITII
Copy link
Member

@IITII IITII commented Sep 2, 2023

#1506

  • VIP 不显示升级条件,hover还是可以正常看到目前符合哪一个等级的要求

  • 对不同的用户组(VIP,管理,普通用户)显示不同的 done 按钮

  • 按通用名称进行了简单的用户分组

    • 管理:admin,moderator,sys,retire,管理,版主,发种,保种,上传,退休
    • VIP:vip,贵宾
  • 预设 icon 从左到右:普通用户,管理,VIP
    image

* VIP 不显示升级条件,hover还是可以正常看到目前符合哪一个等级的要求
* 对不同的用户组(VIP,管理,普通用户)显示不同的 done 按钮
* 目前暂时无法判断 “管理组” 身份,因为存在中英文之分。
@fzlins
Copy link
Collaborator

fzlins commented Sep 2, 2023

用不同的打勾符号有点乱。可以考虑改用户名的名字如黑色,普通会员,红色管理和绿色VIP。或者V符号的代表VIP,人物符号代表管理,以做区分。
还有就是不同站点使用的名称不同,能不能做成可以从config文件里读取。

@IITII
Copy link
Member Author

IITII commented Sep 2, 2023

用不同的打勾符号有点乱。可以考虑改用户名的名字如黑色,普通会员,红色管理和绿色VIP。或者V符号的代表VIP,人物符号代表管理,以做区分。 还有就是不同站点使用的名称不同,能不能做成可以从config文件里读取。

emm,本来的话,nexusphp 应该读 className,在用户名上面有个 className. 规定了不同的等级不同的颜色,这个是统一的。
image

  • 考虑其他架构,好像也没有特别明显一眼能看出来的。

  • 至于不同的用户名对应不同的颜色。感觉没必要,看起来挺混乱的, 而且等级本来就会显示
  • 不同的用户组,用简单的图标代替就行。太多颜色也不和谐

@fzlins
Copy link
Collaborator

fzlins commented Sep 2, 2023 via email

@fzlins
Copy link
Collaborator

fzlins commented Sep 2, 2023

其它架构差别挺大的,所以能从config文件里配置会比较方便。

@IITII IITII requested a review from fzlins September 2, 2023 11:09
@IITII
Copy link
Member Author

IITII commented Sep 2, 2023

之前我也想过换别的 icon,不过试了下之后,感觉差点意思。 不带勾没有灵魂

图标从里面选的,大家也可以提提意见:https://fonts.google.com/icons?selected=Material+Symbols+Outlined:file_download_done:FILL@0;wght@400;GRAD@0;opsz@24&icon.query=check&icon.platform=android

  • 至于 “从 config 文件中配置名称”,需要考虑多语种. 而且一般普通用户没有像 rule.php 一样的地方,能看到所有管理级别的名称。所以我觉得初步配个基本标准的就行。

@IITII
Copy link
Member Author

IITII commented Sep 2, 2023

没有好的设计和思路先简单实现就行,不用纠结那么多,反正也留下了拓展的空间。@fzlins

  • 调整后:管理 - VIP - 普通用户
    image

* 管理:admin,moderator,sys,retire,管理,版主,发种,保种,上传,退休
* VIP:vip,贵宾
@IITII IITII merged commit 66c0248 into pt-plugins:dev Sep 2, 2023
@IITII IITII mentioned this pull request Sep 15, 2023
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