explore page loading fix

Author: feruzm

lib/features/explore/presentation/controller/explore_controller.dart

@@ -16,14 +16,15 @@ class ExploreController extends ChangeNotifier {
   List<TrendingTagModel> tags = [];
   List<TrendingAuthorModel> authors = [];
 
+  bool _authorsLoaded = false;
+
   ExploreController()
       : threadType = getIt<SettingsRepository>().readDefaultThread() {
-    loadData();
+    _loadTags();
   }
 
-  Future<void> loadData() async {
+  Future<void> _loadTags() async {
     tagsState = ViewState.loading;
-    authorsState = ViewState.loading;
     notifyListeners();
 
     final container = _getContainer();
@@ -35,26 +36,47 @@ class ExploreController extends ChangeNotifier {
       tags = [];
       tagsState = ViewState.error;
     }
+    notifyListeners();
+  }
+
+  Future<void> loadAuthorsIfNeeded() async {
+    if (_authorsLoaded) {
+      return;
+    }
+
+    authorsState = ViewState.loading;
+    notifyListeners();
 
+    final container = _getContainer();
     final authorRes = await _repository.getTrendingAuthors(container);
     if (authorRes.isSuccess && authorRes.data != null &&
         authorRes.data!.isNotEmpty) {
       authors = authorRes.data!;
       authorsState = ViewState.data;
+      _authorsLoaded = true;
     } else {
       authors = [];
       authorsState = ViewState.error;
+      _authorsLoaded = false;
     }
     notifyListeners();
   }
 
   void onChangeThreadType(ThreadFeedType type) {
     if (threadType != type) {
       threadType = type;
-      loadData();
+      _resetAuthors();
+      _loadTags();
     }
   }
 
+  void _resetAuthors() {
+    authors = [];
+    authorsState = ViewState.loading;
+    _authorsLoaded = false;
+    notifyListeners();
+  }
+
   String _getContainer() {
     switch (threadType) {
       case ThreadFeedType.ecency:

lib/features/explore/presentation/view/explore_view.dart

@@ -12,43 +12,78 @@ import 'package:waves/features/explore/models/trending_tag_model.dart';
 import 'package:waves/features/explore/presentation/controller/explore_controller.dart';
 import 'package:waves/features/explore/presentation/widgets/thread_type_dropdown.dart';
 
-class ExploreView extends StatelessWidget {
+class ExploreView extends StatefulWidget {
   const ExploreView({super.key});
 
+  @override
+  State<ExploreView> createState() => _ExploreViewState();
+}
+
+class _ExploreViewState extends State<ExploreView>
+    with SingleTickerProviderStateMixin {
+  late final TabController _tabController;
+  late final ExploreController _exploreController;
+
+  @override
+  void initState() {
+    super.initState();
+    _exploreController = ExploreController();
+    _tabController = TabController(length: 2, vsync: this);
+    _tabController.addListener(_onTabChange);
+  }
+
+  @override
+  void dispose() {
+    _tabController.removeListener(_onTabChange);
+    _tabController.dispose();
+    _exploreController.dispose();
+    super.dispose();
+  }
+
+  void _onTabChange() {
+    if (!_tabController.indexIsChanging && _tabController.index == 1) {
+      _exploreController.loadAuthorsIfNeeded();
+    }
+  }
+
   @override
   Widget build(BuildContext context) {
-    return DefaultTabController(
-      length: 2,
-      child: ChangeNotifierProvider(
-        create: (_) => ExploreController(),
-        child: Consumer<ExploreController>(
-          builder: (context, controller, _) {
-            return Scaffold(
-              appBar: AppBar(
-                title: ThreadTypeDropdown(
-                  value: controller.threadType,
-                  onChanged: controller.onChangeThreadType,
-                ),
-                bottom: TabBar(
-                  tabs: [
-                    Tab(text: LocaleText.tags),
-                    Tab(text: LocaleText.users),
-                  ],
-                ),
+    return ChangeNotifierProvider.value(
+      value: _exploreController,
+      child: Consumer<ExploreController>(
+        builder: (context, controller, _) {
+          return Scaffold(
+            appBar: AppBar(
+              title: ThreadTypeDropdown(
+                value: controller.threadType,
+                onChanged: (type) {
+                  controller.onChangeThreadType(type);
+                  if (_tabController.index == 1) {
+                    controller.loadAuthorsIfNeeded();
+                  }
+                },
               ),
-              body: SafeArea(
-                top: false,
-                bottom: false,
-                child: TabBarView(
-                  children: [
-                    _TagsTab(),
-                    _UsersTab(),
-                  ],
-                ),
+              bottom: TabBar(
+                controller: _tabController,
+                tabs: [
+                  Tab(text: LocaleText.tags),
+                  Tab(text: LocaleText.users),
+                ],
               ),
-            );
-          },
-        ),
+            ),
+            body: SafeArea(
+              top: false,
+              bottom: false,
+              child: TabBarView(
+                controller: _tabController,
+                children: [
+                  _TagsTab(),
+                  _UsersTab(),
+                ],
+              ),
+            ),
+          );
+        },
       ),
     );
   }