Skip to content

Add support for androidx.paging#777

Merged
yschimke merged 9 commits into
google:mainfrom
yschimke:paging
Nov 24, 2022
Merged

Add support for androidx.paging#777
yschimke merged 9 commits into
google:mainfrom
yschimke:paging

Conversation

@yschimke

@yschimke yschimke commented Nov 24, 2022

Copy link
Copy Markdown
Collaborator

WHAT

Add support for androidx.paging

WHY

Infinite lists are relevant for message inboxes etc.

HOW

Mostly a port of https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:paging/paging-compose/src/main/java/androidx/paging/compose/LazyPagingItems.kt

Checklist 📋

  • Add explicit visibility modifier and explicit return types for public declarations
  • Run spotless check
  • Run tests
  • Update metalava's signature text files
paging.mp4
@yschimke yschimke requested a review from luizgrp November 24, 2022 15:47

@luizgrp luizgrp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - screenshot/video in the description would be nice

Comment thread sample/src/main/java/com/google/android/horologist/paging/PagingScreen.kt Outdated
Comment thread sample/src/main/java/com/google/android/horologist/sample/MenuChips.kt Outdated
Comment thread sample/src/main/java/com/google/android/horologist/paging/PagingScreen.kt Outdated
Comment thread sample/src/main/java/com/google/android/horologist/paging/PagingScreen.kt Outdated
@yschimke

Copy link
Copy Markdown
Collaborator Author

LGTM - screenshot/video in the description would be nice

done

@luizgrp

luizgrp commented Nov 24, 2022

Copy link
Copy Markdown
Member

nice! the loading indicator seem static in the vid, are the animations disabled in that device?

@yschimke

Copy link
Copy Markdown
Collaborator Author

nice! the loading indicator seem static in the vid, are the animations disabled in that device?

Not sure, I noticed that as well. I'll try on another emulator.

# Conflicts:
#	sample/src/main/java/com/google/android/horologist/sample/SampleWearApp.kt
#	sample/src/main/java/com/google/android/horologist/sample/Screen.kt
@yschimke yschimke self-assigned this Nov 24, 2022
@yschimke

Copy link
Copy Markdown
Collaborator Author

Some issues with placeholders, but that is sample code only.

@yschimke yschimke merged commit b4c5b89 into google:main Nov 24, 2022
@yschimke yschimke deleted the paging branch January 29, 2023 22:10
@yschimke yschimke restored the paging branch January 29, 2023 22:10
@yschimke yschimke deleted the paging branch April 17, 2023 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants