From 555beed47b665bbc0696f3809a84b49ed4fa65e3 Mon Sep 17 00:00:00 2001
From: "Lucas.Xu" <lucas.xu@appflowy.io>
Date: Wed, 17 Aug 2022 20:10:20 +0800
Subject: [PATCH] feat: update tooltips in toolbar

---
 .../src/render/selection/toolbar_widget.dart  | 50 ++++++++++---------
 1 file changed, 27 insertions(+), 23 deletions(-)

diff --git a/frontend/app_flowy/packages/appflowy_editor/lib/src/render/selection/toolbar_widget.dart b/frontend/app_flowy/packages/appflowy_editor/lib/src/render/selection/toolbar_widget.dart
index 2f4eae8f93..19ca0101a9 100644
--- a/frontend/app_flowy/packages/appflowy_editor/lib/src/render/selection/toolbar_widget.dart
+++ b/frontend/app_flowy/packages/appflowy_editor/lib/src/render/selection/toolbar_widget.dart
@@ -96,16 +96,18 @@ class _ToolbarWidgetState extends State<ToolbarWidget> with ToolbarMixin {
           children: [
             // _listToolbar(context),
             // _centerToolbarIcon('divider', width: 10),
-            _centerToolbarIcon('bold'),
-            _centerToolbarIcon('italic'),
-            _centerToolbarIcon('strikethrough'),
-            _centerToolbarIcon('underline'),
+            _centerToolbarIcon('bold', tooltipMessage: 'Bold'),
+            _centerToolbarIcon('italic', tooltipMessage: 'Italic'),
+            _centerToolbarIcon('strikethrough',
+                tooltipMessage: 'Strikethrough'),
+            _centerToolbarIcon('underline', tooltipMessage: 'Underline'),
             _centerToolbarIcon('divider', width: 2),
-            _centerToolbarIcon('quote'),
+            _centerToolbarIcon('quote', tooltipMessage: 'Quote'),
             // _centerToolbarIcon('number_list'),
-            _centerToolbarIcon('bulleted_list'),
+            _centerToolbarIcon('bulleted_list',
+                tooltipMessage: 'Bulleted List'),
             _centerToolbarIcon('divider', width: 2),
-            _centerToolbarIcon('highlight'),
+            _centerToolbarIcon('highlight', tooltipMessage: 'Highlight'),
           ],
         ),
       ),
@@ -122,25 +124,27 @@ class _ToolbarWidgetState extends State<ToolbarWidget> with ToolbarMixin {
   }
 
   Widget _centerToolbarIcon(String name,
-      {Key? key, double? width, VoidCallback? onTap}) {
+      {Key? key, String? tooltipMessage, double? width, VoidCallback? onTap}) {
     return Tooltip(
-      key: key,
-      preferBelow: false,
-      message: name,
-      child: GestureDetector(
-        onTap: onTap ?? () => _onTap(name),
-        child: SizedBox.fromSize(
-          size: width != null
-              ? Size(width, toolbarHeight)
-              : Size.square(toolbarHeight),
-          child: Center(
-            child: FlowySvg(
-              name: 'toolbar/$name',
+        key: key,
+        preferBelow: false,
+        message: tooltipMessage ?? '',
+        child: MouseRegion(
+          cursor: SystemMouseCursors.click,
+          child: GestureDetector(
+            onTap: onTap ?? () => _onTap(name),
+            child: SizedBox.fromSize(
+              size: width != null
+                  ? Size(width, toolbarHeight)
+                  : Size.square(toolbarHeight),
+              child: Center(
+                child: FlowySvg(
+                  name: 'toolbar/$name',
+                ),
+              ),
             ),
           ),
-        ),
-      ),
-    );
+        ));
   }
 
   void _onTapListToolbar(BuildContext context) {