Skip to content

Conversation

@etoledom
Copy link
Contributor

@etoledom etoledom commented Jun 4, 2025

Closes GRA-141

Description

This PR is aimed to solve the issue described on #776 (comment).

The issue was point down to the number of avatars being loaded at once. Solved by using LazyHGrid.

Testing Steps

Steps shown on #776 (comment)

  • On device - dragging down the sheet having many avatars (over 30) should be smooth

@etoledom etoledom requested a review from pinarol June 4, 2025 12:25
@etoledom etoledom self-assigned this Jun 4, 2025
@etoledom etoledom added Bug Something isn't working [Feature] Gravatar-QuickEditor Gravatar Quick Editor [Priority] High labels Jun 4, 2025
@wpmobilebot
Copy link

App Icon📲 You can test the changes from this Pull Request in Gravatar Prototype Build by scanning the QR code below to install the corresponding build.
App NameGravatar Prototype Build
Build Number2416
VersionPR #777
Bundle IDcom.automattic.gravatar-sdk-demo-uikit.prototype-build
Commit7a3eefc
Installation URL171m6brlg9b58
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@pinarol pinarol left a comment

Choose a reason for hiding this comment

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

Looks great. Much smoother! 🚀

ScreenRecording_06-04-2025.14-14-41_1.mp4

@etoledom etoledom merged commit 5b4db04 into release/3.4.0 Jun 4, 2025
9 of 11 checks passed
@etoledom etoledom deleted the etoledom/lazy-h-grid branch June 4, 2025 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working [Feature] Gravatar-QuickEditor Gravatar Quick Editor [Priority] High

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants