Merge pull request Sub6Resources#15 from loitran/feature_link_tap_updated

Sub6Resources · web-flow · commit a80e42983d35 · 2018-08-31T21:45:19.000-06:00
Add link tap callback
diff --git a/README.md b/README.md
@@ -177,4 +177,7 @@ until official support is added.
       padding: EdgeInsets.all(8.0),
       backgroundColor: Colors.white70,
       defaultTextStyle: TextStyle(fontFamily: 'serif'),
+      onLinkTap: (url) {
+        // open url in a webview
+      }
     )
diff --git a/lib/flutter_html.dart b/lib/flutter_html.dart
@@ -4,18 +4,20 @@ import 'package:flutter/material.dart';
 import 'package:flutter_html/html_parser.dart';
 
 class Html extends StatelessWidget {
-  Html(
-      {Key key,
-      @required this.data,
-      this.padding,
-      this.backgroundColor,
-      this.defaultTextStyle = const TextStyle(color: Colors.black)})
-      : super(key: key);
+  Html({
+    Key key,
+    @required this.data,
+    this.padding,
+    this.backgroundColor,
+    this.defaultTextStyle = const TextStyle(color: Colors.black),
+    this.onLinkTap
+  }) : super(key: key);
 
   final String data;
   final EdgeInsetsGeometry padding;
   final Color backgroundColor;
   final TextStyle defaultTextStyle;
+  final Function onLinkTap;
 
   @override
   Widget build(BuildContext context) {
@@ -29,7 +31,7 @@ class Html extends StatelessWidget {
         style: defaultTextStyle,
         child: Wrap(
           alignment: WrapAlignment.start,
-          children: HtmlParser(width: width).parse(data),
+          children: HtmlParser(width: width, onLinkTap: onLinkTap).parse(data),
         ),
       ),
     );
diff --git a/lib/html_parser.dart b/lib/html_parser.dart
@@ -3,9 +3,13 @@ import 'package:html/parser.dart' as parser;
 import 'package:html/dom.dart' as dom;
 
 class HtmlParser {
-  HtmlParser({@required this.width});
+  HtmlParser({
+    @required this.width,
+    this.onLinkTap,
+  });
 
   final double width;
+  final Function onLinkTap;
 
   static const _supportedElements = [
     "a",
@@ -96,14 +100,22 @@ class HtmlParser {
       }
       switch (node.localName) {
         case "a":
-          return DefaultTextStyle.merge(
-            child: Wrap(
-              children: _parseNodeList(node.nodes),
-            ),
-            style: const TextStyle(
-                decoration: TextDecoration.underline,
-                color: Colors.blueAccent,
-                decorationColor: Colors.blueAccent),
+          return GestureDetector(
+            child: DefaultTextStyle.merge(
+              child: Wrap(
+                children: _parseNodeList(node.nodes),
+              ),
+              style: const TextStyle(
+                  decoration: TextDecoration.underline,
+                  color: Colors.blueAccent,
+                  decorationColor: Colors.blueAccent),
+            ),
+            onTap: () {
+              if (node.attributes.containsKey('href') && onLinkTap != null) {
+                String url = node.attributes['href'];
+                onLinkTap(url);
+              }
+            }
           );
         case "abbr":
           return DefaultTextStyle.merge(

Original file line number	Diff line number	Diff line change
`@@ -177,4 +177,7 @@ until official support is added.`
`177`	`177`	`padding: EdgeInsets.all(8.0),`
`178`	`178`	`backgroundColor: Colors.white70,`
`179`	`179`	`defaultTextStyle: TextStyle(fontFamily: 'serif'),`
	`180`	`+ onLinkTap: (url) {`
	`181`	`+ // open url in a webview`
	`182`	`+ }`
`180`	`183`	`)`