Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Fixed changing search solution will cause immediate search behavior #1508

Closed
wants to merge 2 commits into from

Conversation

Meng-Lan
Copy link

描述

  1. 将修改搜索方案从自动保存为配置并且触发搜索更改为不保存配置也不触发修改
  2. 主动触发搜索才保存当前选择的搜索方案为默认搜索方案
  3. 添加dev build的环境变量,否则有报错

@Meng-Lan Meng-Lan changed the title Fixed changing search solution will cause immediate search behavior Fix: Fixed changing search solution will cause immediate search behavior Jul 29, 2023
@ted423
Copy link
Collaborator

ted423 commented Sep 1, 2023

有点没看懂

@Meng-Lan
Copy link
Author

Meng-Lan commented Sep 1, 2023

已知有搜索方案”电视剧“、”电影“、”动漫”。
假如你在搜索完“消失的她”(电影)之后,你想搜索“斗破苍穹”(动漫)

  1. 先切换搜索方案,再输入“斗破苍穹”:会出现直接针对动漫进行“消失的她”进行搜索,然后你输入完斗破苍穹之后回车,第二次搜索
  2. 先输入“斗破苍穹”,再切换搜索方案:行为还是一样,切换的时候使用的搜索词是“消失的她”。因为你输入的搜索词还没被saveConfig

会导致的问题也就是短时间内进行多次搜索,只需要搜索10次变成了15次,然后有些网站就会限制搜索次数,结果返回为空

@IITII
Copy link
Member

IITII commented Sep 1, 2023

有点没太看懂. 相对于搜索方案的保存, #257 说的这个问题才是关键吧?

  • 而且都手动切换搜索方案了, 不保存方案明显有问题吧. =.=

@Meng-Lan
Copy link
Author

Meng-Lan commented Sep 1, 2023

  1. 这个修改解决了他提到的那个“立即触发搜索并且使用旧关键字”的bug
  2. 他提到的使用新关键字搜索也不make sense,因为要求先修改关键字后修改方案,反过来一样是个bug
  3. 这次修改在切换方案后没有保存,需要执行搜索才进行saveConfig是一个workaround

直接执行saveConfig会更奇怪,如果监听solution会导致自动搜索,而不监听,我如果搜相同词条”斗破苍穹“从动画换成电视剧,由于key没变,导致不会响应搜索。
但我觉得现在的方式至少比上面提到的都会更加合理,又不需要花更多的精力去解决保存方案修改

@IITII
Copy link
Member

IITII commented Sep 1, 2023

重新看了一下。他说的确实不太明确。
当前的搜索个人比较介意的一点是:搜索后想切换别的方案并修改别的关键字进行搜索。但是在切换方案后,直接触发了搜索。

想调整成只有输入框回车时才触发搜索。

@IITII
Copy link
Member

IITII commented Sep 1, 2023

切换后立即搜索,更符合很多相似的搜索方案的场景
切换,手动触发,更符合差异比较大的搜索方案

个人更偏向后者。因为方案多了,切起来也麻烦,不如写一起。

@Meng-Lan
Copy link
Author

Meng-Lan commented Sep 1, 2023

主要还是key写在watch里,如果changeSolutionId会saveConfig但不执行doSearch,那么针对key不变,solution变的情况下是没法trigger的。这个问题应该是可以加一个监听回车的event,然后执行doSearch,但前提是要保存一个<key,solutionId>来check,防止相同<key,solutionId>多次搜索。

@IITII IITII assigned IITII and unassigned IITII Sep 2, 2023
@IITII
Copy link
Member

IITII commented Sep 5, 2023

方式不对, 应该调整 solution 监听那块. 我待会改下吧.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants