Skip to content

Commit 8eaff06

Browse files
committed
fix(ui): adjust entity details page layout and error handling
- Remove errorBuilder from Image widget to prevent icon display on image load failure - Replace errorBuilder with an empty SizedBox for a cleaner UI - Adjust padding for SliverPadding widget to remove bottom padding - Restrict alternative UI for non-source entities without appBarIconData - Optimize icon placement padding in SliverAppBar
1 parent dd5f7c1 commit 8eaff06

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

lib/entity_details/view/entity_details_page.dart

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -192,15 +192,12 @@ class _EntityDetailsViewState extends State<EntityDetailsView> {
192192
width: AppSpacing.xxl,
193193
height: AppSpacing.xxl,
194194
fit: BoxFit.contain,
195-
errorBuilder: (context, error, stackTrace) => Icon(
196-
appBarIconData ?? Icons.info_outline,
197-
size: AppSpacing.xxl,
198-
color: colorScheme.onSurfaceVariant,
199-
),
195+
errorBuilder: (context, error, stackTrace) =>
196+
const SizedBox(),
200197
),
201198
),
202199
)
203-
else if (appBarIconData != null)
200+
else if (state.entity is Source && appBarIconData != null)
204201
Padding(
205202
padding: Directionality.of(context) == TextDirection.ltr
206203
? const EdgeInsets.only(right: AppSpacing.md)
@@ -267,8 +264,10 @@ class _EntityDetailsViewState extends State<EntityDetailsView> {
267264
)
268265
else
269266
SliverPadding(
270-
padding: const EdgeInsets.symmetric(
271-
horizontal: AppSpacing.paddingMedium,
267+
padding: const EdgeInsets.only(
268+
top: AppSpacing.paddingMedium,
269+
left: AppSpacing.paddingMedium,
270+
right: AppSpacing.paddingMedium,
272271
),
273272
sliver: SliverList.separated(
274273
itemCount:

0 commit comments

Comments
 (0)