#1 - 2020-11-5 14:38
Cedar (。´-д-)
现在的slidebar的sticky状态好像是用JS配合CSS实现的, 问题多多..
比如如果主楼有图片而且图片加载速度太慢(或者用了我的折叠组件)的话, 就会导致sliderbar在错误的位置变成sticky状态..
而且变成sticky状态的瞬间, 评论区会因为缺了这个元素立刻上移..
其实现在用CSS很容易就能实现这个效果, 只要加上
#sliderContainer {
  position: -webkit-sticky;
  position: sticky;
  top: 0;
}
就可以了..甚至不需要用JS监听事件..

=========

目前想到两种打补丁的方法:
1. 跑一下 $(window).off('scroll'); 去掉监听事件, 然后加样式.
不过这个监听添加的时间好像很晚, 好像要等#sliderContainer出现后才能去掉..可能还要配合MutationObserver之类的..有点麻烦..

2. 直接修改CSS, 加上上面提到的样式后,
给 #sliderContainer.sticky 加上一个优先级更高的CSS,
覆盖掉原来的 position: fixed; 效果..
这样相当于让事件失效..
虽然简单, 但是就真的是"补丁"操作了..

所以还是让sai老板改改代码比较好..
#2 - 2020-11-5 19:45
(夏花绚烂,星河璀璨。)
资瓷,顶(bgm37)
#3 - 2020-11-5 20:02
(DD雷达搜寻中...?)
谢谢 我甚至不知道position还能用sticky(bgm38)
#3-1 - 2020-11-5 21:04
Cedar
还有一个很简单的CSS..
奇偶数楼层用 :nth-child(odd) 或者 :nth-child(even) 就能区分了..没想到bangumi的实现方式竟然不是这样的..
#3-2 - 2020-11-5 21:15
chitanda@Lv2
Cedar 说: 还有一个很简单的CSS..
奇偶数楼层用 :nth-child(odd) 或者 :nth-child(even) 就能区分了..没想到bangumi的实现方式竟然不是这样的..
考虑一下BGM第一版是12年前。。。
#3-3 - 2020-11-5 21:21
Cedar
chitanda@Lv2 说: 考虑一下BGM第一版是12年前。。。
嘛..但是这个优化福利这么大, 真的没更新过嘛..
#4 - 2020-11-9 01:30
基本在 PC 浏览器上看,下滑不像触控屏那样随意连续,以至于几乎感知不到这个问题,细微拖动才注意到确实有元素脱离导致抖动(bgm38)
#4-1 - 2020-11-9 08:34
Cedar
有的, 图片加载速度比较慢的时候非常明显
#5 - 2020-11-9 01:53
(大変に気分がいい)
我都是油猴脚本优化2333
自己定样式啥的
毕竟本来就有几个样式问题
#5-1 - 2020-11-9 08:37
Cedar
这些小问题自己打补丁差别不大却有点拖慢速度..怎么都不太痛快的样子..
#5-2 - 2020-11-10 00:31
桜小路ルナ
Cedar 说: 这些小问题自己打补丁差别不大却有点拖慢速度..怎么都不太痛快的样子..
等是等不到的 又不开源 你可以搞钛合金插件啊
#5-3 - 2020-11-10 01:00
Cedar
桜小路ルナ 说: 等是等不到的 又不开源 你可以搞钛合金插件啊
搞是容易搞, 但是不爽啊, 这种小改动肯定是官方维护比较好..
#5-4 - 2020-11-10 16:36
Cedar
桜小路ルナ 说: 等是等不到的 又不开源 你可以搞钛合金插件啊
突然想起来我之前写过一个BBCode效果补丁的钛合金组件, 然而到现在也没过审..
#5-5 - 2020-11-10 17:08
🦋喵小六❄️
Cedar 说: 突然想起来我之前写过一个BBCode效果补丁的钛合金组件, 然而到现在也没过审..
可能是打算合并到master去,但是忙着忙着就忘了XD
#5-6 - 2020-11-10 18:35
Cedar
🦋喵小六❄️ 说: 可能是打算合并到master去,但是忙着忙着就忘了XD
我之前在控制台发现有一条CSS是 ul{list-style: none},
说明多级列表的样式是故意被设置成none的..

往坏了想是sai老板不想加这个功能..
往好了想是当年偷懒了, 没专门适配用户评论..
或者打算将来直接更新一个功能更完整的BBCode..(但是先鸽个5年再说