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

Search string is emphasized (bold) in UI #812

Merged
merged 1 commit into from
Jun 15, 2024
Merged

Search string is emphasized (bold) in UI #812

merged 1 commit into from
Jun 15, 2024

Conversation

bolsinga
Copy link
Owner

  • RankingList now defers rendering LibraryComparable.name to a closure
  • ArtistList and VenueList will render their names emphasizing the searchString in the UI.

- RankingList now defers rendering LibraryComparable.name to a closure
- ArtistList and VenueList will render their names emphasizing the searchString in the UI.
@bolsinga bolsinga merged commit 5b9ff19 into main Jun 15, 2024
1 check passed
@bolsinga bolsinga deleted the EmphasizeSearch branch June 15, 2024 17:56
bolsinga added a commit that referenced this pull request Jun 16, 2024
Basically Typing "G " for "Going Well" would have the markdown of "Goin**g **Well" and this does not render properly.

Revert "Search string is emphasized (bold) in UI (#812)"

This reverts commit 5b9ff19.

Revert "fix when emphasizing the empty string (#811)"

This reverts commit 552914b.

Revert "add utility the emphasize case insensitive portions of strings (#810)"

This reverts commit 86d6b3d.
@bolsinga bolsinga mentioned this pull request Jun 16, 2024
bolsinga added a commit that referenced this pull request Jun 16, 2024
- This restores #810, #811, #812, and fixes an issue when rendering found that inspired me to back out #815.
- A night's sleep found this solution.
bolsinga added a commit that referenced this pull request Jun 16, 2024
- This restores #810, #811, #812, and fixes an issue when rendering
found that inspired me to back out with #815.
- A night's sleep found this 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.

None yet

1 participant