-
Notifications
You must be signed in to change notification settings - Fork 42
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
検索履歴の追加 #2202
検索履歴の追加 #2202
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2202 +/- ##
==========================================
+ Coverage 41.01% 41.08% +0.06%
==========================================
Files 148 148
Lines 3757 3763 +6
Branches 587 599 +12
==========================================
+ Hits 1541 1546 +5
- Misses 2214 2215 +1
Partials 2 2
Continue to review full report at Codecov.
|
- todo 削除したときは`store.currentInput`に追加されないでほしい
:class="$style.icon" | ||
:size="24" | ||
:inner-size="12" | ||
@click.stop="removeHistorySuggestion(label)" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここで消すのではなくて、親にremoveイベント送って親で消してほしいです 🙏
これは選択のほうもそういう実装になってるので一貫性を保ちたいからです
src/components/Main/CommandPalette/SearchSuggestionHistoryItem.vue
Outdated
Show resolved
Hide resolved
src/components/Main/CommandPalette/SearchSuggestionHistoryItem.vue
Outdated
Show resolved
Hide resolved
….vue Co-authored-by: 翠 / green <[email protected]>
Co-authored-by: 翠 / green <[email protected]>
Co-authored-by: 翠 / green <[email protected]>
Co-authored-by: 翠 / green <[email protected]>
Co-authored-by: 翠 / green <[email protected]>
- 一部変数名変更
….vue Co-authored-by: 翠 / green <[email protected]>
….vue Co-authored-by: 翠 / green <[email protected]>
修正しました |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
デグレってます
ほかはよさそうです 👍
src/components/Main/CommandPalette/SearchSuggestionHistoryItem.vue
Outdated
Show resolved
Hide resolved
….vue Co-authored-by: 翠 / green <[email protected]>
直しました:pray: |
src/components/Main/CommandPalette/SearchSuggestionHistoryItem.vue
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:yosa: 🎉
履歴が表示されるところまでは作りましたが、履歴が更新されません。