gaogaotiantian / biliscope

Bilibili chrome extension to show uploader's stats
MIT License
575 stars 45 forks source link

Add profession to user card #210

Closed F-park closed 1 month ago

F-park commented 1 month ago

B站自带

image

此 PR 实现

image

https://space.bilibili.com/2990100


close #209

gaogaotiantian commented 1 month ago

这个feature本身没有任何问题,但是细节处理的部分,去比较是不是' '太别扭了,不要这么比。这个titledescription是什么关系?它俩是有可能出现任何一个么?理想情况下我们肯定是可以通过data['profession']直接判断是不是需要显示的,而不是检查它是不是一个带一个空格的字符串。如果它俩可以出现任意一个,那就意味着你的string可能前面或者后面带一个空格啊。

gaogaotiantian commented 1 month ago

我没太懂你的方向。你这里应该是,data['profession']在有的情况下,是需要显示的string,在没有的情况下,是个空string,或者null(任何evaluate成false的东西)。也就是在读数据那里做处理。

F-park commented 1 month ago

应该是department,数据以title department显示,里面有一个is_show标识,我用那个来判断了。

gaogaotiantian commented 1 month ago

那就是this.data["profession"] = d["data"]["is_show"] ?xxx: '';