-
Notifications
You must be signed in to change notification settings - Fork 75
Feature: Search bar for subtitles #91
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
Conversation
Hi @KRSHH, Thanks PR!
I have not tried it, but it might be better to implement it on the View side using the filtering function of CollectionView. https://wpf-tutorial.com/listview-control/listview-filtering/ I use this for search feature in the cheat sheet. LLPlayer/LLPlayer/ViewModels/CheatSheetDialogVM.cs Lines 73 to 100 in 323c834
Or it may be easier to jump to a hit subtitle without filtering like PotPlayer. |
Hi @umlx5h thanks for pointing out the issues. I have addressed them in this commit. and also, how would text highlighting look in the search section? and can you explain the "PotPlayer" feature... |
Thanks, this one is better than the old one. Please remove the old changes as they are still there.
Do you mean the highlighting of the text in the hit subtitle? This seems difficult to implement, so you don't have to do it.
PotPlayer's subtitle browser jumps back and forth instead of filtering when searching. This method is probably more difficult to implement. So the one that is easier to implement is OK. I want to achieve the following requirements by the time of release.
|
As a user, who is trying to learn a language through videos. I usually watch 5-6 hour long documentaries or movies. And navigating thru that by hitting and scrolling would be very difficult and would take lots of time. The user sometimes might also miss the part while hitting... So that's an extra annoyance... This is my opinion from the user perspective. |
- Create a view in the secondary subtitle as well - Filtering using displayed text - CTRL+F as global key - Remove old changes - Use behaivor for getting focus of TextBox - Some performance optimization - Use async/await to debounce
Hi @KRSHH, there were many codes I wanted to change, so I fixed them for my taste. |
Hey, the changes were perfectly reasonable. Thanks. |
Added a search bar to search text in subtitles and navigate through the video easily
