fix offline webview. (#295)

Livinglist · Nov 3, 2023 · 109b928 · 109b928
1 parent 939d55e
commit 109b928
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 10 deletions.
diff --git a/lib/extensions/context_extension.dart b/lib/extensions/context_extension.dart
@@ -85,4 +85,8 @@ extension ContextExtension on BuildContext {
   int get storyTileMaxLines {
     return _storyTileMaxLines;
   }
+
+  double get topPadding {
+    return MediaQuery.of(this).padding.top + kToolbarHeight;
+  }
 }
diff --git a/lib/screens/item/item_screen.dart b/lib/screens/item/item_screen.dart
@@ -216,8 +216,6 @@ class _ItemScreenState extends State<ItemScreen>
 
   @override
   Widget build(BuildContext context) {
-    final double topPadding =
-        MediaQuery.of(context).padding.top + kToolbarHeight;
     return BlocBuilder<AuthBloc, AuthState>(
       builder: (BuildContext context, AuthState authState) {
         return MultiBlocListener(
@@ -273,7 +271,7 @@ class _ItemScreenState extends State<ItemScreen>
                             scrollOffsetListener: scrollOffsetListener,
                             commentEditingController: commentEditingController,
                             authState: authState,
-                            topPadding: topPadding,
+                            topPadding: context.topPadding,
                             splitViewEnabled: widget.splitViewEnabled,
                             onMoreTapped: onMoreTapped,
                             onRightMoreTapped: onRightMoreTapped,
@@ -347,7 +345,7 @@ class _ItemScreenState extends State<ItemScreen>
                       scrollOffsetListener: scrollOffsetListener,
                       commentEditingController: commentEditingController,
                       authState: authState,
-                      topPadding: topPadding,
+                      topPadding: context.topPadding,
                       splitViewEnabled: widget.splitViewEnabled,
                       onMoreTapped: onMoreTapped,
                       onRightMoreTapped: onRightMoreTapped,

diff --git a/lib/screens/web_view/web_view_screen.dart b/lib/screens/web_view/web_view_screen.dart
@@ -20,6 +20,7 @@ class WebViewScreen extends StatefulWidget {
 
 class _WebViewScreenState extends State<WebViewScreen> {
   final WebViewController controller = WebViewController();
+  bool showFullUrl = false;
 
   @override
   void initState() {
@@ -43,15 +44,26 @@ class _WebViewScreenState extends State<WebViewScreen> {
     return Scaffold(
       appBar: AppBar(
         backgroundColor: Theme.of(context).canvasColor,
-        title: Text(
-          humanize(widget.url),
-          style: const TextStyle(
-            fontSize: TextDimens.pt12,
+        foregroundColor: Theme.of(context).colorScheme.onSurface,
+        title: GestureDetector(
+          onTap: () {
+            setState(() {
+              showFullUrl = !showFullUrl;
+            });
+          },
+          child: Text(
+            showFullUrl
+                ? humanize(widget.url)
+                : Uri.parse(widget.url).authority,
+            style: const TextStyle(
+              fontSize: TextDimens.pt14,
+            ),
+            overflow: TextOverflow.ellipsis,
+            textAlign: TextAlign.center,
           ),
-          overflow: TextOverflow.ellipsis,
-          textAlign: TextAlign.center,
         ),
         centerTitle: true,
+        elevation: 0,
       ),
       body: WebViewWidget(
         controller: controller,

diff --git a/lib/screens/widgets/offline_banner.dart b/lib/screens/widgets/offline_banner.dart
@@ -22,6 +22,7 @@ class OfflineBanner extends StatelessWidget {
       builder: (BuildContext context, StoriesState state) {
         if (state.isOfflineReading) {
           return MaterialBanner(
+            dividerColor: Palette.transparent,
             content: Text(
               'You are currently in offline mode. '
               '${showExitButton ? 'Exit to fetch latest stories.' : ''}',