gaogaotiantian / biliscope

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

feat: Add `minSize` configuration item #36

Closed 14Kay closed 1 year ago

14Kay commented 1 year ago

新增了 minSize可选配置,并随手写了个UI

14Kay commented 1 year ago

你新做的这个option UI可以截个图上来看一下嘛?

QQ截图20230527211137 没有设计 就随便写的 XD

gaogaotiantian commented 1 year ago

这两个按钮是不是太大了一点……哈哈。注意一下我前面还有个comment,好像没看到。

14Kay commented 1 year ago

这两个按钮是不是太大了一点……哈哈。注意一下我前面还有个comment,好像没看到。

你这么说 还真觉得

gaogaotiantian commented 1 year ago

还是有个comment没改,前面有关minSize: minSize的。然后你删掉了两个空行么?我是想让你把“空格”删了,就是保留那个空行,但是不要额外加空格。

在做PR的时候,尽量不要去碰到和你的PR功能无关的原来的一些style的事情,除非你专门就是PR改style的。

gaogaotiantian commented 1 year ago

@14Kay 你还想继续做这个PR么?你把我上面那个comment给address之后我就可以merge了。因为我也有一个需要改options的PR,我怕我先merge了我的你这个会conflict。所以如果你还想merge,我就等你一下。

14Kay commented 1 year ago

@14Kay 你还想继续做这个PR么?你把我上面那个comment给address之后我就可以merge了。因为我也有一个需要改options的PR,我怕我先merge了我的你这个会conflict。所以如果你还想merge,我就等你一下。

好的,周末玩游戏给忘记了 我以为已经合了

gaogaotiantian commented 1 year ago

428和430这两行,你现在这个line上有空格……你能不能保持一个空的空行,就是这一行保留,但是这一行没有内容,你现在有一大堆空格,所以它们还会显示在diff里。理论上这里如果你没有动,就不会显示在diff里

14Kay commented 1 year ago

428和430这两行,你现在这个line上有空格……你能不能保持一个空的空行,就是这一行保留,但是这一行没有内容,你现在有一大堆空格,所以它们还会显示在diff里。理论上这里如果你没有动,就不会显示在diff里

懂了 这个应该是我编码用的LF的问题

gaogaotiantian commented 1 year ago

428和430这两行,你现在这个line上有空格……你能不能保持一个空的空行,就是这一行保留,但是这一行没有内容,你现在有一大堆空格,所以它们还会显示在diff里。理论上这里如果你没有动,就不会显示在diff里

懂了 这个应该是我编码用的LF的问题

这两行应该明显是你在按回车的时候somehow编辑器自动补了空格……然后你没把空格删了……

gaogaotiantian commented 1 year ago

你把endline改了,不要改endline……我说了就是空格的问题不是你LF的问题,现在整个file都diff掉了。你在commit的时候要先看一下diff,确认一下这些是你要做的diff……

14Kay commented 1 year ago

你把endline改了,不要改endline……我说了就是空格的问题不是你LF的问题,现在整个file都diff掉了。你在commit的时候要先看一下diff,确认一下这些是你要做的diff……

对不起啊 编辑器里确实没显示那么多diff XD

gaogaotiantian commented 1 year ago

感谢贡献!

14Kay commented 1 year ago

感谢贡献!

学到了很多!下次提PR会更加严谨