Android icon indicating copy to clipboard operation
Android copied to clipboard

[Bug] Text search of bookmarks crashes app

Open trailblazingfive opened this issue 3 years ago • 3 comments

Describe the bug

Having a huge bookmark list +5000 records, after latest update 5.137.5 app crashes when trying to do a text search - error message. Downgraded to 5.135.0, and imported bookmarks to the app (previously exported from it), but out of 1MB file only ~300kB imported, rest got dropped.

How to Reproduce

Go to bookmarks Click search Start inserting text App crashes - error message

Expected behavior

Expected to get search results.

Environment

- DDG App Version: 5.137.5
- Device: Pocophone F1
- OS: Android 9

trailblazingfive avatar Oct 01 '22 21:10 trailblazingfive

Hey @trailblazingfive can you assign this bug to me ? I can help with this

dinesh-thiyagarajan avatar Oct 27 '22 08:10 dinesh-thiyagarajan

This seems to be fixed.

There was some line overriding the size, but ListAdapter seems to do diffing async so we have to let ListAdapter give the getItemCount, as the diff might not be ready yet. The file's been renamed BookmarkFolderStructureAdapter, and the override has been removed.

https://github.com/duckduckgo/Android/blob/d46883eda1ca176d5555ec9b15e809db99438070/app/src/main/java/com/duckduckgo/app/bookmarks/ui/bookmarkfolders/BookmarkFoldersAdapter.kt#L81

bemusementpark avatar Aug 11 '24 02:08 bemusementpark