v1.1.0: Major refactoring and improvements
This commit is contained in:
@@ -1,5 +1,4 @@
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter/services.dart';
|
||||
|
||||
class SimpleCountriesSidebar extends StatelessWidget {
|
||||
final List<String> countries;
|
||||
@@ -25,21 +24,10 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
print('🔥 SIDEBAR BUILD ============================================');
|
||||
print('🔥 SIDEBAR BUILD - Countries count: ${countries.length}');
|
||||
print('🔥 SIDEBAR BUILD - Is Loading: $isLoading');
|
||||
print('🔥 SIDEBAR BUILD - Is Organizing: $isOrganizing');
|
||||
print('🔥 SIDEBAR BUILD - Countries list: $countries');
|
||||
print('🔥 SIDEBAR BUILD - Selected country: "$selectedCountry"');
|
||||
|
||||
if (countries.isNotEmpty) {
|
||||
print('🔥 SIDEBAR BUILD - First 10 countries:');
|
||||
for (int i = 0; i < countries.length && i < 10; i++) {
|
||||
print('🔥 SIDEBAR BUILD [${i + 1}] "${countries[i]}"');
|
||||
}
|
||||
for (int i = 0; i < countries.length && i < 10; i++) {}
|
||||
}
|
||||
print('🔥 SIDEBAR BUILD ============================================');
|
||||
|
||||
|
||||
return Container(
|
||||
width: 250,
|
||||
color: Colors.grey[900],
|
||||
@@ -64,7 +52,7 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
],
|
||||
),
|
||||
),
|
||||
|
||||
|
||||
// List
|
||||
Expanded(
|
||||
child: isOrganizing || (isLoading && countries.isEmpty)
|
||||
@@ -82,25 +70,25 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
),
|
||||
)
|
||||
: countries.isEmpty
|
||||
? const Center(
|
||||
child: Text(
|
||||
'No hay países',
|
||||
style: TextStyle(color: Colors.white54),
|
||||
),
|
||||
)
|
||||
: ListView.builder(
|
||||
padding: const EdgeInsets.symmetric(vertical: 8),
|
||||
itemCount: _getItemCount(),
|
||||
itemBuilder: (context, index) {
|
||||
return _buildItemAtIndex(context, index);
|
||||
},
|
||||
),
|
||||
? const Center(
|
||||
child: Text(
|
||||
'No hay países',
|
||||
style: TextStyle(color: Colors.white54),
|
||||
),
|
||||
)
|
||||
: ListView.builder(
|
||||
padding: const EdgeInsets.symmetric(vertical: 8),
|
||||
itemCount: _getItemCount(),
|
||||
itemBuilder: (context, index) {
|
||||
return _buildItemAtIndex(context, index);
|
||||
},
|
||||
),
|
||||
),
|
||||
],
|
||||
),
|
||||
);
|
||||
}
|
||||
|
||||
|
||||
Widget _buildCountryItem(String name, bool isSelected, VoidCallback onTap) {
|
||||
return FocusableActionDetector(
|
||||
actions: <Type, Action<Intent>>{
|
||||
@@ -125,10 +113,16 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
child: Container(
|
||||
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
|
||||
decoration: BoxDecoration(
|
||||
color: isSelected ? Colors.red : (hasFocus ? Colors.red.withValues(alpha: 0.5) : Colors.transparent),
|
||||
color: isSelected
|
||||
? Colors.red
|
||||
: (hasFocus
|
||||
? Colors.red.withValues(alpha: 0.5)
|
||||
: Colors.transparent),
|
||||
border: Border(
|
||||
left: BorderSide(
|
||||
color: isSelected ? Colors.white : (hasFocus ? Colors.white : Colors.transparent),
|
||||
color: isSelected
|
||||
? Colors.white
|
||||
: (hasFocus ? Colors.white : Colors.transparent),
|
||||
width: 4,
|
||||
),
|
||||
),
|
||||
@@ -167,7 +161,9 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
|
||||
// Find insertion point for "Fútbol Argentino" (after Perú)
|
||||
final peruIndex = countries.indexOf('Perú');
|
||||
final footballInsertIndex = peruIndex >= 0 ? peruIndex + 1 : countries.length;
|
||||
final footballInsertIndex = peruIndex >= 0
|
||||
? peruIndex + 1
|
||||
: countries.length;
|
||||
|
||||
if (showFootballCategory) {
|
||||
// Adjust for "Todos" at index 0 and "Fútbol Argentino" after Perú
|
||||
@@ -231,21 +227,23 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
child: Container(
|
||||
padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 12),
|
||||
decoration: BoxDecoration(
|
||||
color: isSelected ? Colors.green[700] : (hasFocus ? Colors.green[700]?.withOpacity(0.8) : Colors.green[900]?.withOpacity(0.3)),
|
||||
color: isSelected
|
||||
? Colors.green[700]
|
||||
: (hasFocus
|
||||
? Colors.green[700]?.withValues(alpha: 0.8)
|
||||
: Colors.green[900]?.withValues(alpha: 0.3)),
|
||||
border: Border(
|
||||
left: BorderSide(
|
||||
color: isSelected ? Colors.white : (hasFocus ? Colors.white : Colors.green[400]!),
|
||||
color: isSelected
|
||||
? Colors.white
|
||||
: (hasFocus ? Colors.white : Colors.green[400]!),
|
||||
width: 4,
|
||||
),
|
||||
),
|
||||
),
|
||||
child: Row(
|
||||
children: [
|
||||
Icon(
|
||||
Icons.sports_soccer,
|
||||
color: Colors.white,
|
||||
size: 20,
|
||||
),
|
||||
Icon(Icons.sports_soccer, color: Colors.white, size: 20),
|
||||
const SizedBox(width: 12),
|
||||
Expanded(
|
||||
child: Text(
|
||||
@@ -253,7 +251,9 @@ class SimpleCountriesSidebar extends StatelessWidget {
|
||||
style: TextStyle(
|
||||
color: Colors.white,
|
||||
fontSize: 14,
|
||||
fontWeight: isSelected ? FontWeight.bold : FontWeight.normal,
|
||||
fontWeight: isSelected
|
||||
? FontWeight.bold
|
||||
: FontWeight.normal,
|
||||
),
|
||||
),
|
||||
),
|
||||
|
||||
Reference in New Issue
Block a user