gaogaotiantian / biliscope

Bilibili chrome extension to show uploader's stats
MIT License
594 stars 46 forks source link

Improve style #79

Closed Redlnn closed 1 year ago

Redlnn commented 1 year ago

这个 PR 做出了如下改动:

  1. 个人资料页的备注区在 hover 状态时使用B站品牌色作为边框颜色
  2. 悬浮卡片上 UP 主标签的背景颜色使用B站品牌色,之前的颜色比较深暗,在整个页面中的观感稍差
  3. 调整了标签的 padding 和 margin 距离,占用面积更小,多行标签时行与行之间有适当的间隔
调整前 调整后
image image
gaogaotiantian commented 1 year ago

能不能发一个单行tag的对比图我看一下?

Redlnn commented 1 year ago

能不能发一个单行tag的对比图我看一下?

单行的效果图:

image

Redlnn commented 1 year ago

虽然 Force Push 不太好,但是我个人觉得多用一两个 commit 把改动改回去有点污染 commit 历史这样子,而且并不是在主仓库里,不知道高天对于这种情况一般是在 fork 出来的仓库/新分支里 Force Push 呢还是用新的 commit 覆盖呢?

gaogaotiantian commented 1 year ago

feature branch的history是没有意义的,都会squash掉,想怎么搞怎么搞。无论是乱七八糟的commit,还是因为rebase或者其他原因的force push,都是可以接受的,只要PR的最终结果对就行。