NewPipe icon indicating copy to clipboard operation
NewPipe copied to clipboard

Use ListAdapter for search predictions.

Open Isira-Seneviratne opened this issue 2 years ago • 1 comments

What is it?

  • [ ] Bugfix (user facing)
  • [x] Feature (user facing)
  • [x] Codebase improvement (dev facing)
  • [ ] Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Use ListAdapter to handle displaying list data in PeertubeInstanceListFragment, SuggestionListAdapter and PreferenceSearchAdapter.

Before/After Screenshots/Screen Record

  • Before:
    • Video search: https://user-images.githubusercontent.com/31027858/182021881-f60beea1-e880-42f8-a9d2-173b8ea8f72d.mp4
    • Preference search: https://user-images.githubusercontent.com/31027858/182053004-86985d79-852b-4dfd-a3f0-1df1b83c7c4b.mp4
  • After:
    • Video search: https://user-images.githubusercontent.com/31027858/182021890-30818b27-d39d-43d5-91d0-8d041227225a.mp4
    • Preference search: https://user-images.githubusercontent.com/31027858/182053021-971da415-3bb9-491e-a986-b297d48bbc91.mp4

Fixes the following issue(s)

  • Fixes #

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.

Due diligence

Isira-Seneviratne avatar Jul 31 '22 10:07 Isira-Seneviratne

Sorry, I took a small break over the past week to relax after the past semester.

Could this be rebased to remove the dev merge commit?

TacoTheDank avatar Aug 11 '22 19:08 TacoTheDank

Sorry, I took a small break over the past week to relax after the past semester.

Could this be rebased to remove the dev merge commit?

Sure.

Isira-Seneviratne avatar Aug 11 '22 22:08 Isira-Seneviratne

@Stypox Are we doing a feature freeze until v0.24.0 is released, or can this be merged now?

TacoTheDank avatar Aug 12 '22 21:08 TacoTheDank

No, we have not decided upon a freeze, you can proceed @TacoTheDank

Stypox avatar Aug 16 '22 18:08 Stypox

Thanks for this feature!

TacoTheDank avatar Aug 19 '22 19:08 TacoTheDank