[SMS App] Improve filter
ClosedPublic

Authored by sredman on Dec 16 2018, 12:10 AM.

Details

Summary
  • Allow filter box to handle keystrokes passed to the ListView
    • This is a little cheesy, but fixes T8341
  • Pin filter box to always be visible
  • Clear filter on Esc
Test Plan
  • Try typing a contact you would like to search for
    • Before: Every time you type a new letter, the filter box would lose focus and you would have to click it to give it focus again
    • Now: You can type smoothly
  • Try scrolling the list
    • Before: The filter box disappeared off the top of the screen
    • Now: The filter box is always visible
  • Try pressing escape
    • Before: Nothing
    • Now: Filter clears
  • Bonus: When using the arrow keys to navigate, before the filter was manually handling those and the view would not loop. Now, the view loops if you try to select an item past the beginning or end (Easiest to see with highlighting from D17612)

Diff Detail

Repository
R224 KDE Connect
Branch
smsapp-filter
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6062
Build 6080: arc lint + arc unit
sredman created this revision.Dec 16 2018, 12:10 AM
Restricted Application added a project: KDE Connect. · View Herald TranscriptDec 16 2018, 12:10 AM
Restricted Application added a subscriber: kdeconnect. · View Herald Transcript
sredman requested review of this revision.Dec 16 2018, 12:10 AM
apol accepted this revision.Dec 17 2018, 2:29 AM
This revision is now accepted and ready to land.Dec 17 2018, 2:29 AM
sredman closed this revision.Dec 17 2018, 4:26 PM