fix(mobile): add keys for person tiles in search (#27689)
* fix(mobile): key for person tiles in search * chore: add key to avatar * chore: use simple personId * chore: rename key in person pagepull/27133/merge
parent
8fb2c7755d
commit
3b47ca1c37
|
|
@ -89,7 +89,7 @@ class _DriftPeopleCollectionPageState extends ConsumerState<DriftPeopleCollectio
|
|||
shape: const CircleBorder(side: BorderSide.none),
|
||||
elevation: 3,
|
||||
child: CircleAvatar(
|
||||
key: ValueKey('avatar-${person.id}'),
|
||||
key: ValueKey(person.id),
|
||||
maxRadius: isTablet ? 100 / 2 : 96 / 2,
|
||||
backgroundImage: RemoteImageProvider(url: getFaceThumbnailUrl(person.id)),
|
||||
),
|
||||
|
|
|
|||
|
|
@ -57,6 +57,7 @@ class PeoplePicker extends HookConsumerWidget {
|
|||
final isSelected = selectedPeople.value.contains(person);
|
||||
|
||||
return Padding(
|
||||
key: ValueKey(person.id),
|
||||
padding: const EdgeInsets.only(bottom: 2.0),
|
||||
child: LargeLeadingTile(
|
||||
title: Text(
|
||||
|
|
@ -73,6 +74,7 @@ class PeoplePicker extends HookConsumerWidget {
|
|||
shape: const CircleBorder(side: BorderSide.none),
|
||||
elevation: 3,
|
||||
child: CircleAvatar(
|
||||
key: ValueKey(person.id),
|
||||
maxRadius: imageSize / 2,
|
||||
backgroundImage: RemoteImageProvider(url: getFaceThumbnailUrl(person.id)),
|
||||
),
|
||||
|
|
|
|||
Loading…
Reference in New Issue