refactor: avatar loading

Author: stefanJi

lib/ui/activity/activity_tab.dart

@@ -1,6 +1,7 @@
 import 'dart:convert';
 
 import 'package:F4Lab/gitlab_client.dart';
+import 'package:F4Lab/util/widget_util.dart';
 import 'package:F4Lab/widget/comm_ListView.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/widgets.dart';
@@ -41,9 +42,7 @@ class FeedState extends CommListState<TabActivity> {
     final item = data[index];
     return Card(
       child: ListTile(
-        leading: CircleAvatar(
-          backgroundImage: NetworkImage(item['avatar']),
-        ),
+        leading: loadAvatar(item['avatar'], item['title']),
         title: Text(item['title']),
         onTap: () {},
       ),

lib/ui/project/project_tabs.dart

@@ -1,5 +1,6 @@
 import 'package:F4Lab/model/project.dart' as model;
 import 'package:F4Lab/ui/project/project_detail.dart';
+import 'package:F4Lab/util/widget_util.dart';
 import 'package:F4Lab/widget/comm_ListView.dart';
 import 'package:flutter/material.dart';
 
@@ -41,20 +42,11 @@ class ProjectState extends CommListState<ProjectTab> {
   @override
   Widget childBuild(BuildContext context, int index) {
     final item = model.Project.fromJson(data[index]);
+    final name = item.name;
+    final color = Theme.of(context).primaryColor;
     return Card(
       child: ListTile(
-        leading: item.avatarUrl != null
-            ? CircleAvatar(
-                backgroundImage: NetworkImage(item.avatarUrl),
-                backgroundColor: Theme.of(context).primaryColor,
-              )
-            : CircleAvatar(
-                backgroundColor: Theme.of(context).primaryColor,
-                child: Text(
-                  item.name.substring(0, 1).toUpperCase(),
-                  style: TextStyle(color: Theme.of(context).accentColor),
-                ),
-              ),
+        leading: loadAvatar(item.avatarUrl ?? null, name, color: color),
         title: Text(item.nameWithNamespace),
         subtitle: Text(item.description ?? item.lastActivityAt),
         trailing: item.defaultBranch != null

lib/ui/todo/todo_tab.dart

@@ -1,4 +1,5 @@
 import 'package:F4Lab/model/todo.dart' as TodoModel;
+import 'package:F4Lab/util/widget_util.dart';
 import 'package:F4Lab/widget/comm_ListView.dart';
 import 'package:flutter/material.dart';
 
@@ -13,9 +14,7 @@ class TodoState extends CommListState<TabTodo> {
     final todoItem = TodoModel.Todo.fromJson(data[index]);
     return Card(
         child: ExpansionTile(
-      leading: CircleAvatar(
-        backgroundImage: NetworkImage(todoItem.author.avatarUrl),
-      ),
+      leading: loadAvatar(todoItem.author.avatarUrl, todoItem.author.name),
       title: Text.rich(TextSpan(
           text: "${todoItem.author.name} ",
           style: TextStyle(fontWeight: FontWeight.w100),

lib/util/widget_util.dart

@@ -3,21 +3,20 @@ import 'package:flutter/material.dart';
 Widget loadAvatar(String url, String name, {Color color = Colors.teal}) {
   assert(name != null);
   if (url != null) {
+    debugPrint("[loadAvatar] Start load: $url");
     NetworkImage image;
-    try {
-      image = new NetworkImage(url);
-      return CircleAvatar(
-        backgroundImage: image,
-      );
-    } catch (Exception) {
-      print("Load avatar error: $Exception");
-    }
+    image = new NetworkImage(url);
+    return CircleAvatar(
+      backgroundImage: image,
+      backgroundColor: color,
+    );
   }
   return new CircleAvatar(
     child: Text(
       name,
       textAlign: TextAlign.center,
-      overflow: TextOverflow.fade,
+      overflow: TextOverflow.ellipsis,
+      softWrap: true,
     ),
     backgroundColor: color,
   );