Closed F-park closed 10 months ago
首先你需要rebase一下啊,有conflict。
其次,啥时候发明了@
这个符号了……不要去加一些压根没有的功能啊。就biliscope整个体系里完全没有和@
有关的事情,就不要去碰它……这是一个不太好的习惯。
你现在这个solution是ok的,但是有一个问题,就是用户再也切不回来了,想从用户切到别的tab都会被你再给转回来。无论用户的intention是什么样的,这种behavior都是个大问题,很容易让用户觉得这里出什么bug了。
所以这边啊,你要把这个redirect做成一次性的。在检查的时候,就稍微strict点,只redirect all
这个path的(默认的搜索path)。然后跳转的时候在那一堆argument里加一个,比如biliscope_redirected=true
之类的,然后再看到有这个的就不redirect了。这样就可以做成一个一次性的跳转。
首先你需要rebase一下啊,有conflict。
其次,啥时候发明了
@
这个符号了……不要去加一些压根没有的功能啊。就biliscope整个体系里完全没有和@
有关的事情,就不要去碰它……这是一个不太好的习惯。你现在这个solution是ok的,但是有一个问题,就是用户再也切不回来了,想从用户切到别的tab都会被你再给转回来。无论用户的intention是什么样的,这种behavior都是个大问题,很容易让用户觉得这里出什么bug了。
所以这边啊,你要把这个redirect做成一次性的。在检查的时候,就稍微strict点,只redirect
all
这个path的(默认的搜索path)。然后跳转的时候在那一堆argument里加一个,比如biliscope_redirected=true
之类的,然后再看到有这个的就不redirect了。这样就可以做成一个一次性的跳转。
redirect 就是一次性的,因为这个跳转没有写到 observer 里,只有页面加载时才会触发,我都测试过的,不会产生奇怪的跳转。不过严格判断 /all
这个可以加上。
白名单永远好过黑名单,所以把检查/upuser
起码变成/all
吧。跳转的话好像确实B站不刷新页面的,没事。把@
的部分删了就ok了。
白名单永远好过黑名单,所以把检查
/upuser
起码变成/all
吧。跳转的话好像确实B站不刷新页面的,没事。把@
的部分删了就ok了。
才发现我的判断逻辑有问题,应该判断 /all
就好了,现在改好了。
新功能描述
现在在搜索词前缀为
#
时会自动跳转用户搜索界面(只会在第一次进页面时触发)Resolve #95