refactor: improve error message handling in profile settings and add width constraints to agent tag chips

This commit is contained in:
pradeepkumar
2026-05-06 15:35:34 +05:30
parent 019df719fc
commit 0be0da7c57
2 changed files with 70 additions and 48 deletions

View File

@@ -1113,24 +1113,37 @@ class _AgentHomeContentState extends ConsumerState<_AgentHomeContent> {
isExpanded ? tags : tags.take(initialCount).toList(); isExpanded ? tags : tags.take(initialCount).toList();
final remaining = tags.length - initialCount; final remaining = tags.length - initialCount;
// Cap each chip's width so long taglines wrap inside the chip instead
// of overflowing the Wrap's row and clipping behind ancestor bounds.
return LayoutBuilder(
builder: (ctx, constraints) {
final maxChipWidth = constraints.maxWidth.isFinite
? constraints.maxWidth
: MediaQuery.of(ctx).size.width - 32;
return Wrap( return Wrap(
spacing: 8, spacing: 8,
runSpacing: 8, runSpacing: 8,
children: [ children: [
...displayTags.map((tag) => Container( ...displayTags.map((tag) => ConstrainedBox(
padding: constraints: BoxConstraints(maxWidth: maxChipWidth),
const EdgeInsets.symmetric(horizontal: 12, vertical: 6), child: Container(
padding: const EdgeInsets.symmetric(
horizontal: 12, vertical: 6),
decoration: BoxDecoration( decoration: BoxDecoration(
borderRadius: BorderRadius.circular(15), borderRadius: BorderRadius.circular(15),
border: border: Border.all(
Border.all(color: AppColors.primaryDark, width: 0.7), color: AppColors.primaryDark, width: 0.7),
), ),
child: Text(tag, child: Text(
tag,
softWrap: true,
style: const TextStyle( style: const TextStyle(
fontFamily: 'SourceSerif4', fontFamily: 'SourceSerif4',
fontSize: 14, fontSize: 14,
fontWeight: FontWeight.w500, fontWeight: FontWeight.w500,
color: AppColors.primaryDark)), color: AppColors.primaryDark),
),
),
)), )),
if (remaining > 0) if (remaining > 0)
GestureDetector( GestureDetector(
@@ -1145,11 +1158,12 @@ class _AgentHomeContentState extends ConsumerState<_AgentHomeContent> {
}, },
behavior: HitTestBehavior.opaque, behavior: HitTestBehavior.opaque,
child: Container( child: Container(
padding: padding: const EdgeInsets.symmetric(
const EdgeInsets.symmetric(horizontal: 12, vertical: 6), horizontal: 12, vertical: 6),
decoration: BoxDecoration( decoration: BoxDecoration(
borderRadius: BorderRadius.circular(15), borderRadius: BorderRadius.circular(15),
border: Border.all(color: AppColors.primaryDark, width: 0.1), border: Border.all(
color: AppColors.primaryDark, width: 0.1),
), ),
child: Text( child: Text(
isExpanded ? 'Show Less' : '+$remaining More', isExpanded ? 'Show Less' : '+$remaining More',
@@ -1162,6 +1176,8 @@ class _AgentHomeContentState extends ConsumerState<_AgentHomeContent> {
), ),
], ],
); );
},
);
} }
// ── Info Cards (Availability, Work Env, Testimonial) — matches web ── // ── Info Cards (Availability, Work Env, Testimonial) — matches web ──

View File

@@ -6,6 +6,7 @@ import 'package:flutter/services.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart';
import 'package:image_picker/image_picker.dart'; import 'package:image_picker/image_picker.dart';
import 'package:real_estate_mobile/core/constants/app_colors.dart'; import 'package:real_estate_mobile/core/constants/app_colors.dart';
import 'package:real_estate_mobile/core/network/api_exceptions.dart';
import 'package:real_estate_mobile/core/utils/image_url_resolver.dart'; import 'package:real_estate_mobile/core/utils/image_url_resolver.dart';
import 'package:real_estate_mobile/core/widgets/s3_image.dart'; import 'package:real_estate_mobile/core/widgets/s3_image.dart';
import 'package:real_estate_mobile/features/profile/data/profile_repository.dart'; import 'package:real_estate_mobile/features/profile/data/profile_repository.dart';
@@ -500,7 +501,12 @@ class _ChangeEmailSheetState extends State<_ChangeEmailSheet> {
if (!mounted) return; if (!mounted) return;
setState(() { setState(() {
_submitting = false; _submitting = false;
_localError = e.toString().replaceFirst('Exception: ', ''); // ApiException carries the backend's message verbatim
// (e.g. "Incorrect password", "Email is already in use").
// Fall back to a generic message for unexpected error types.
_localError = e is ApiException
? e.message
: 'Something went wrong. Please try again.';
}); });
} }
} }