State restorable fortnightly (flutter#390)

* State restorable fortnightly
thkim1011 · Dec 17, 2020 · c8734e9 · c8734e9
1 parent a950e14
commit c8734e9
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/lib/layout/image_placeholder.dart b/lib/layout/image_placeholder.dart
@@ -63,9 +63,7 @@ class FadeInImagePlaceholder extends StatelessWidget {
       height: height,
       fit: fit,
       frameBuilder: (context, child, frame, wasSynchronouslyLoaded) {
-        // We only need to animate the loading of images on the web. We can
-        // also return early if the images are already in the cache.
-        if (wasSynchronouslyLoaded || !kIsWeb) {
+        if (wasSynchronouslyLoaded) {
           return this.child ?? child;
         } else {
           return AnimatedSwitcher(

diff --git a/lib/studies/fortnightly/app.dart b/lib/studies/fortnightly/app.dart
@@ -24,6 +24,7 @@ class FortnightlyApp extends StatelessWidget {
         ? const _FortnightlyHomeDesktop()
         : const _FortnightlyHomeMobile();
     return MaterialApp(
+      restorationScopeId: 'fortnightly_app',
       title: _fortnightlyTitle,
       debugShowCheckedModeBanner: false,
       theme: buildTheme(context).copyWith(
@@ -76,6 +77,7 @@ class _FortnightlyHomeMobile extends StatelessWidget {
       ),
       body: SafeArea(
         child: ListView(
+          restorationId: 'list_view',
           children: [
             HashtagBar(),
             for (final item in buildArticlePreviewItems(context))

diff --git a/lib/studies/fortnightly/shared.dart b/lib/studies/fortnightly/shared.dart
@@ -244,6 +244,7 @@ class HashtagBar extends StatelessWidget {
     return SizedBox(
       height: height,
       child: ListView(
+        restorationId: 'hashtag_bar_list_view',
         scrollDirection: Axis.horizontal,
         children: [
           const SizedBox(width: 16),