Apps-android-commons: Clearing recent searches only removes last 10

Created on 20 Dec 2018  路  2Comments  路  Source: commons-app/apps-android-commons

Summary:

Currently, clicking the 'bin' icon on searches only deletes your last 10 searches. Personally I think it should wipe your entire search history - I'm not sure why it's implemented the way it is currently.

If however we do want it to only delete last 10 then at least the method call should be renamed to something other than deleteAll

https://github.com/commons-app/apps-android-commons/blob/b0b4b0810084a89d2be80c10814be921ecaa4835/app/src/main/java/fr/free/nrw/commons/explore/recentsearches/RecentSearchesDao.java#L58

assigned bug

Most helpful comment

Hey.I had started working on this

All 2 comments

I agree that it should delete the entire search history

Hey.I had started working on this

Was this page helpful?
0 / 5 - 0 ratings

Related issues

nicolas-raoul picture nicolas-raoul  路  56Comments

misaochan picture misaochan  路  59Comments

97balakrishnan picture 97balakrishnan  路  60Comments

misaochan picture misaochan  路  64Comments

nicolas-raoul picture nicolas-raoul  路  59Comments