Skip to content

Commit 7404d63

Browse files
committed
refactor(discover): simplify source card layout
- Remove outer Padding widget - Adjust Card margin for spacing - Simplify InkWell child layout
1 parent 5888721 commit 7404d63

File tree

1 file changed

+33
-40
lines changed

1 file changed

+33
-40
lines changed

lib/discover/view/discover_page.dart

Lines changed: 33 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -272,50 +272,43 @@ class _SourceCard extends StatelessWidget {
272272

273273
return SizedBox(
274274
width: 150,
275-
child: Padding(
276-
padding: const EdgeInsets.symmetric(horizontal: AppSpacing.sm),
277-
child: Card(
278-
clipBehavior: Clip.antiAlias,
279-
child: InkWell(
280-
onTap: () => context.pushNamed(
281-
Routes.entityDetailsName,
282-
pathParameters: {
283-
'type': ContentType.source.name,
284-
'id': source.id,
285-
},
286-
),
287-
child: Column(
288-
mainAxisAlignment: MainAxisAlignment.center,
289-
children: [
290-
Expanded(
291-
child: Padding(
292-
padding: const EdgeInsets.all(AppSpacing.md),
293-
child: Image.network(
294-
source.logoUrl,
295-
fit: BoxFit.contain,
296-
errorBuilder: (context, error, stackTrace) => Icon(
297-
Icons.source_outlined,
298-
size: AppSpacing.xxl,
299-
color: theme.colorScheme.onSurfaceVariant,
300-
),
275+
child: Card(
276+
margin: const EdgeInsets.symmetric(horizontal: AppSpacing.sm),
277+
clipBehavior: Clip.antiAlias,
278+
child: InkWell(
279+
onTap: () => context.pushNamed(
280+
Routes.entityDetailsName,
281+
pathParameters: {'type': ContentType.source.name, 'id': source.id},
282+
),
283+
child: Column(
284+
mainAxisAlignment: MainAxisAlignment.center,
285+
children: [
286+
Expanded(
287+
child: Padding(
288+
padding: const EdgeInsets.all(AppSpacing.md),
289+
child: Image.network(
290+
source.logoUrl,
291+
fit: BoxFit.contain,
292+
errorBuilder: (context, error, stackTrace) => Icon(
293+
Icons.source_outlined,
294+
size: AppSpacing.xxl,
295+
color: theme.colorScheme.onSurfaceVariant,
301296
),
302297
),
303298
),
304-
Padding(
305-
padding: const EdgeInsets.symmetric(
306-
horizontal: AppSpacing.xs,
307-
),
308-
child: Text(
309-
source.name,
310-
style: theme.textTheme.bodyMedium,
311-
textAlign: TextAlign.center,
312-
maxLines: 1,
313-
overflow: TextOverflow.ellipsis,
314-
),
299+
),
300+
Padding(
301+
padding: const EdgeInsets.symmetric(horizontal: AppSpacing.xs),
302+
child: Text(
303+
source.name,
304+
style: theme.textTheme.bodyMedium,
305+
textAlign: TextAlign.center,
306+
maxLines: 1,
307+
overflow: TextOverflow.ellipsis,
315308
),
316-
const SizedBox(height: AppSpacing.sm),
317-
],
318-
),
309+
),
310+
const SizedBox(height: AppSpacing.sm),
311+
],
319312
),
320313
),
321314
),

0 commit comments

Comments
 (0)