search

adrian / upm-android

Android version of Universal Password Manager.
http://upm.sourceforge.net/
76 stars 45 forks source link

[Search] Add AutoCompleteTextView for search #20

Closed moeadham closed 3 years ago

moeadham commented 10 years ago

Search was somewhat cumbersome requiring over two screen touches, and did not perform AutoComplete as per Andoid usage styling. I decided to add basic AutoComplete support.

This commit DOES NOT remove the old search functionality. This may be a good idea, please let me know and I can commit a second change to remove old functionality.

doubleo2 commented 9 years ago

@adrian Is there a reason this hasn't been merged? Is upm-android still active and do you accept pull requests?

adrian commented 9 years ago

Hi @moeadham. Sorry for not replying to you earlier. Your PR looks good. As you mention yourself it probably makes sense to remove the old search feature. It feels a bit unnecessary to have two ways of searching.