search

hanydd / BilibiliSponsorBlock

一款跳过B站视频中恰饭片段的浏览器插件，移植自 SponsorBlock。A browser extension to skip sponsored segments in videos on Bilibili.com, ported from the SponsorBlock
https://www.bsbsb.top
GNU General Public License v3.0
1.97k stars 48 forks source link

Increase the functionality of the YT icon #33

Closed NeKoOuO closed 2 months ago

NeKoOuO commented 2 months ago
NeKoOuO commented 2 months ago

@hanydd 頭疼 沒有衝突 結果構建出來 測試的時候按鈕消失不見 求救~

hanydd commented 2 months ago

啊这不是出来了吗？是不是启动项目出问题了 image

hanydd commented 2 months ago

这动画可以的，很强👍

NeKoOuO commented 2 months ago

啊这不是出来了吗？是不是启动项目出问题了 image

有出來就好 應該是我的環境問題XD

hanydd commented 2 months ago

不错不错，就剩下几个小问题：

  1. 还是感觉红色确实有点太显眼了…要不加一个灰色调，和旁边的按钮能融为一体？
  2. 图标的大小不太对，这里最高应该不会超过24px
  3. 记得把弃用的css从content.css里面删掉，要不真成垃圾场了（应该就#bsbPortButton和#bsbPortButtonImage这两个） 我个人感觉在插件里用id作为css selector更好一点，可以避免污染到页面其他的element，冲突了也好debug。

（cleanup.ts里面也会对插件向页面插入的元素做一个清理，但是这部分我还没发现到底有没有用，所以不是很重要） https://github.com/hanydd/BilibiliSponsorBlock/blob/68e5dc387e8299f6f5e3fdfa1bb7a04ff4fbf738/src/utils/cleanup.ts#L103-L125

NeKoOuO commented 2 months ago

23好了檢查一下 然後你說的灰色調沒懂 能不能講具體點 要不你直接review change吧

hanydd commented 2 months ago

没事，先合并了，我再慢慢找颜色

NeKoOuO commented 2 months ago

真的好怪哦 我這新版本真的沒有icon image

hanydd commented 2 months ago

重复不出来啊，刷新页面也没用吗？

hanydd commented 2 months ago

是不是黑色模式的原因？打开F12看下元素在吗？

NeKoOuO commented 2 months ago

重复不出来啊，刷新页面也没用吗？

因為我之前多開帳號用一個新的環境測試 結果還是一樣顯示不出來

NeKoOuO commented 2 months ago

是不是黑色模式的原因？打开F12看下元素在吗？

好像真有可能是這個原因

NeKoOuO commented 2 months ago

啊这不是出来了吗？是不是启动项目出问题了 image

看你的環境確實是亮色模式

hanydd commented 2 months ago

看你的環境確實是亮色模式

一般我测试都是直接npm run dev启动一个纯净的Chrome

NeKoOuO commented 2 months ago

恩....亮色的狀況好像一樣 一開始就沒有這個元素 image