goodjoblife / GoodJobShare

GoodJob 職場透明化運動
https://www.goodjob.life/
MIT License
82 stars 11 forks source link

search query變動時不改變捲軸位置 #1441

Closed peteranny closed 3 weeks ago

peteranny commented 1 month ago

Close #1439

這個 PR 是?

原本,location有任何變化時,就會重設捲軸位置。

但搜尋經驗時產生的 ?q 變化,並不適合重設捲軸位置。

本 PR 將 ?q 作為例外。

Screenshots

https://github.com/user-attachments/assets/c2168c09-a81e-45b7-970f-293abe0c68d2

我應該如何手動測試?

mark86092 commented 1 month ago

?q 作為例外是指 搜尋經驗 這個頁面嗎,還是適用站上全部頁面有 ?q ?

因為有幾個頁面會有 ?q 的產生:

不確定這個 PR 會不會對他們有影響

peteranny commented 1 month ago

?q 作為例外是指 搜尋經驗 這個頁面嗎,還是適用站上全部頁面有 ?q ?

會的,全站都有影響

效果就是在公司搜尋頁面,如果換搜尋文字,捲軸一樣不會置頂。

如果將這個行為與公司頁面的搜尋視為相同的情境,我自己覺得可以接受。

可以看 @barry800414 覺得如何?

barry800414 commented 1 month ago

?q 作為例外是指 搜尋經驗 這個頁面嗎,還是適用站上全部頁面有 ?q ?

會的,全站都有影響

效果就是在公司搜尋頁面,如果換搜尋文字,捲軸一樣不會置頂。

如果將這個行為與公司頁面的搜尋視為相同的情境,我自己覺得可以接受。

可以看 @barry800414 覺得如何?

@mark86092 @peteranny

我試了幾個主要用搜尋框的情境:

  1. 任何頁面用 header 的搜尋框
  2. 公司的薪資/面試/評價列表頁面 用職稱搜索資料
  3. 職稱的薪資/面試/評價列表頁面 用公司搜索資料

使用的體驗沒有什麼問題。我自己覺得是可以 go 的 當然,確實是這是一個在 codebase 裡面比較隱晦的邏輯(?q= 的時候不會觸發捲軸捲動),所以確實以後有可能遇到非預期行為 但我想像我們短期內也不會有新的搜尋相關的功能