fix: 调整搜索方案默认搜索逻辑, 可在 通用配置-搜索 中调整 #1561
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
按我的理解，这只是不执行doSearch的逻辑，会saveConfig。 |
|
|
对，所有我想说的是，切换了搜索方案之后，不改关键词应该是没法执行搜索，这是个bug。 |
IITII
commented
Sep 7, 2023
Comment on lines
+406
to
+409
| if (key === this.searchKey && this.$router.currentRoute.name === targetRoute.name) { | ||
| console.log(`skip same searchTorrent: key: ${key}, searchKey: ${this.searchKey}`) | ||
| return; | ||
| } |
There was a problem hiding this comment.
这里判了 route name, 你可以在这里通过自己加 event 触发搜索. 目前过滤掉就好了 @Meng-Lan
反正直接刷新网页, 刷新后会搜索
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix: 调整搜索方案默认搜索逻辑, 可在 通用配置-搜索 中调整
#257
#1508
截图