feat: refactor agent filter sheet to support dynamic API-driven types and update professional card layout
This commit is contained in:
@@ -96,6 +96,11 @@ class _AgentSearchScreenState extends ConsumerState<AgentSearchScreen> {
|
|||||||
onClearAll: () {
|
onClearAll: () {
|
||||||
ref.read(searchAgentsProvider.notifier).clearFilters();
|
ref.read(searchAgentsProvider.notifier).clearFilters();
|
||||||
},
|
},
|
||||||
|
onListingTypeChange: (agentTypeId) {
|
||||||
|
ref
|
||||||
|
.read(searchAgentsProvider.notifier)
|
||||||
|
.filterByAgentType(agentTypeId);
|
||||||
|
},
|
||||||
),
|
),
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -10,6 +10,7 @@ class AgentFilterSheet extends StatefulWidget {
|
|||||||
final String? selectedAgentTypeId;
|
final String? selectedAgentTypeId;
|
||||||
final void Function(Map<String, List<String>> filters) onApply;
|
final void Function(Map<String, List<String>> filters) onApply;
|
||||||
final VoidCallback onClearAll;
|
final VoidCallback onClearAll;
|
||||||
|
final void Function(String? agentTypeId)? onListingTypeChange;
|
||||||
|
|
||||||
const AgentFilterSheet({
|
const AgentFilterSheet({
|
||||||
super.key,
|
super.key,
|
||||||
@@ -19,6 +20,7 @@ class AgentFilterSheet extends StatefulWidget {
|
|||||||
this.selectedAgentTypeId,
|
this.selectedAgentTypeId,
|
||||||
required this.onApply,
|
required this.onApply,
|
||||||
required this.onClearAll,
|
required this.onClearAll,
|
||||||
|
this.onListingTypeChange,
|
||||||
});
|
});
|
||||||
|
|
||||||
@override
|
@override
|
||||||
@@ -28,6 +30,7 @@ class AgentFilterSheet extends StatefulWidget {
|
|||||||
class _AgentFilterSheetState extends State<AgentFilterSheet> {
|
class _AgentFilterSheetState extends State<AgentFilterSheet> {
|
||||||
late Map<String, List<String>> _filters;
|
late Map<String, List<String>> _filters;
|
||||||
final Map<String, bool> _expandedSections = {};
|
final Map<String, bool> _expandedSections = {};
|
||||||
|
String? _selectedAgentTypeId;
|
||||||
|
|
||||||
@override
|
@override
|
||||||
void initState() {
|
void initState() {
|
||||||
@@ -37,6 +40,7 @@ class _AgentFilterSheetState extends State<AgentFilterSheet> {
|
|||||||
for (final entry in widget.currentFilters.entries) {
|
for (final entry in widget.currentFilters.entries) {
|
||||||
_filters[entry.key] = List<String>.from(entry.value);
|
_filters[entry.key] = List<String>.from(entry.value);
|
||||||
}
|
}
|
||||||
|
_selectedAgentTypeId = widget.selectedAgentTypeId;
|
||||||
// Expand all sections by default
|
// Expand all sections by default
|
||||||
for (final field in widget.filterFields) {
|
for (final field in widget.filterFields) {
|
||||||
_expandedSections[field.slug] = true;
|
_expandedSections[field.slug] = true;
|
||||||
@@ -231,33 +235,47 @@ class _AgentFilterSheetState extends State<AgentFilterSheet> {
|
|||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
List<Widget> _buildListingTypeChips() {
|
/// Pluralize an agent type name for display (e.g. "Agent" → "Agents",
|
||||||
// Find "Professional" and "Lender" agent types
|
/// "Lender" → "Lenders", "Property Manager" → "Property Managers").
|
||||||
final types = <_ListingChip>[];
|
/// Falls back to the raw name if it's already plural-looking.
|
||||||
for (final at in widget.agentTypes) {
|
String _displayLabel(String name) {
|
||||||
final name = at.name.toLowerCase();
|
final trimmed = name.trim();
|
||||||
if (name == 'professional' || name == 'agent') {
|
if (trimmed.isEmpty) return trimmed;
|
||||||
types.add(_ListingChip(label: 'Agents', id: at.id));
|
final lower = trimmed.toLowerCase();
|
||||||
} else if (name == 'lender') {
|
if (lower.endsWith('s')) return trimmed; // already plural
|
||||||
types.add(_ListingChip(label: 'Lenders', id: at.id));
|
if (lower.endsWith('y')) {
|
||||||
|
return '${trimmed.substring(0, trimmed.length - 1)}ies';
|
||||||
}
|
}
|
||||||
}
|
return '${trimmed}s';
|
||||||
if (types.isEmpty) {
|
|
||||||
types.add(const _ListingChip(label: 'Agents', id: null));
|
|
||||||
types.add(const _ListingChip(label: 'Lenders', id: null));
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return types.map((chip) {
|
List<Widget> _buildListingTypeChips() {
|
||||||
final isSelected = chip.id == widget.selectedAgentTypeId;
|
// Render one chip per backend agent type — fully driven by API data
|
||||||
|
if (widget.agentTypes.isEmpty) return const [];
|
||||||
|
|
||||||
|
return widget.agentTypes.map((at) {
|
||||||
|
final isSelected = at.id == _selectedAgentTypeId;
|
||||||
return Padding(
|
return Padding(
|
||||||
padding: const EdgeInsets.only(left: 12),
|
padding: const EdgeInsets.only(left: 12),
|
||||||
|
child: GestureDetector(
|
||||||
|
behavior: HitTestBehavior.opaque,
|
||||||
|
onTap: () {
|
||||||
|
setState(() {
|
||||||
|
// Toggle off when tapping the already-selected chip
|
||||||
|
_selectedAgentTypeId = isSelected ? null : at.id;
|
||||||
|
});
|
||||||
|
widget.onListingTypeChange?.call(_selectedAgentTypeId);
|
||||||
|
},
|
||||||
child: Text(
|
child: Text(
|
||||||
chip.label,
|
_displayLabel(at.name),
|
||||||
style: TextStyle(
|
style: TextStyle(
|
||||||
fontFamily: 'SourceSerif4',
|
fontFamily: 'SourceSerif4',
|
||||||
fontSize: 18,
|
fontSize: 18,
|
||||||
fontWeight: isSelected ? FontWeight.w500 : FontWeight.w400,
|
fontWeight: isSelected ? FontWeight.w700 : FontWeight.w400,
|
||||||
color: AppColors.primaryDark,
|
color: isSelected
|
||||||
|
? AppColors.accentOrange
|
||||||
|
: AppColors.primaryDark,
|
||||||
|
),
|
||||||
),
|
),
|
||||||
),
|
),
|
||||||
);
|
);
|
||||||
@@ -322,12 +340,6 @@ class _AgentFilterSheetState extends State<AgentFilterSheet> {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
class _ListingChip {
|
|
||||||
final String label;
|
|
||||||
final String? id;
|
|
||||||
const _ListingChip({required this.label, required this.id});
|
|
||||||
}
|
|
||||||
|
|
||||||
class _FilterOptionsList extends StatefulWidget {
|
class _FilterOptionsList extends StatefulWidget {
|
||||||
final FilterableField field;
|
final FilterableField field;
|
||||||
final bool Function(String value) isSelected;
|
final bool Function(String value) isSelected;
|
||||||
|
|||||||
@@ -105,7 +105,7 @@ class _TopProfessionalsSectionState
|
|||||||
else ...[
|
else ...[
|
||||||
// Professional Cards - horizontal scroll
|
// Professional Cards - horizontal scroll
|
||||||
SizedBox(
|
SizedBox(
|
||||||
height: 430,
|
height: 540,
|
||||||
child: ListView.builder(
|
child: ListView.builder(
|
||||||
controller: _scrollController,
|
controller: _scrollController,
|
||||||
scrollDirection: Axis.horizontal,
|
scrollDirection: Axis.horizontal,
|
||||||
@@ -295,7 +295,6 @@ class _TopProfessionalsSectionState
|
|||||||
final rating = item.rating > 0 ? item.rating : 5.0;
|
final rating = item.rating > 0 ? item.rating : 5.0;
|
||||||
|
|
||||||
return Container(
|
return Container(
|
||||||
constraints: const BoxConstraints(maxHeight: 410),
|
|
||||||
decoration: BoxDecoration(
|
decoration: BoxDecoration(
|
||||||
color: Colors.white,
|
color: Colors.white,
|
||||||
borderRadius: BorderRadius.circular(15),
|
borderRadius: BorderRadius.circular(15),
|
||||||
@@ -335,7 +334,10 @@ class _TopProfessionalsSectionState
|
|||||||
),
|
),
|
||||||
|
|
||||||
// Content
|
// Content
|
||||||
Padding(
|
Flexible(
|
||||||
|
child: SingleChildScrollView(
|
||||||
|
physics: const NeverScrollableScrollPhysics(),
|
||||||
|
child: Padding(
|
||||||
padding: const EdgeInsets.fromLTRB(16, 12, 16, 14),
|
padding: const EdgeInsets.fromLTRB(16, 12, 16, 14),
|
||||||
child: Column(
|
child: Column(
|
||||||
crossAxisAlignment: CrossAxisAlignment.start,
|
crossAxisAlignment: CrossAxisAlignment.start,
|
||||||
@@ -534,6 +536,8 @@ class _TopProfessionalsSectionState
|
|||||||
],
|
],
|
||||||
),
|
),
|
||||||
),
|
),
|
||||||
|
),
|
||||||
|
),
|
||||||
],
|
],
|
||||||
),
|
),
|
||||||
);
|
);
|
||||||
|
|||||||
Reference in New Issue
Block a user