Skip to content

Feature/search highlight#168

Open
johntheprime wants to merge 4 commits intoOmGodse:masterfrom
johntheprime:feature/search-highlight
Open

Feature/search highlight#168
johntheprime wants to merge 4 commits intoOmGodse:masterfrom
johntheprime:feature/search-highlight

Conversation

@johntheprime
Copy link
Copy Markdown

Hi, Om

When searching for items, I found that when note contents are with more than 20 lines, it's very difficult to target the searched content instantly. This pr would highlight the content that's searched.

Note: the last pr's compile errors fix is included here.

Add search term highlighting when viewing notes

  • Pass search keyword when navigating from search results
  • Highlight matching text in title, body, and list items
  • Skip BackgroundColorSpan when saving to prevent persistence
  • Clear search keyword when leaving search view

- Convert textSize string to float in bindLabels call
- Fix pin lambda to call bindPinned instead of shadowed variable
- Pass search keyword when navigating from search results
- Highlight matching text in title, body, and list items
- Skip BackgroundColorSpan when saving to prevent persistence
- Clear search keyword when leaving search view
- Remove createBaseNote call from setState
- Only save note when content is not empty
- Handle new note creation in saveNote method
- Swipe across notes to select/deselect them
- Start selection mode by swiping past 30% of an item width
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.

1 participant