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

Add the ability to search and find text in the current track #39

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sirken
Copy link

@sirken sirken commented May 27, 2024

Hello, I've enjoyed using your project and find it very useful, thanks for creating it.

One feature I found myself missing was the ability to do a text search to find lyrics instead of visually scrolling to find the location, so I created a basic find function. The keys are similar to the less or vim style search and can be changed via the config. If you're interested in using any of the code, here it is.

In the PR are the following updates:

New find function

  • Added the ability to find text
  • Matches in each line are highlighted
  • Ability to progress to the [n]ext or [p]revious line match
  • Keys can be changed via the config file

Other updates

  • Added a status bar with track scroll percentage
  • Status bar can be turned on/off via the config file
  • Changed the track so it can scroll to 100% and be compatible with the find functionality
  • Moved [a]uto scroll and [d]elete track status items flush left to be compatible with percentage position

Regards!

Add lyrics search functionality and a track percentage status bar. Enables scrolling 100% to the bottom of the track so that search and manual scroll are compatible
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.

None yet

1 participant