From eeb6b69f58bd9a1bd93d3d3c48d9d75d80da96d8 Mon Sep 17 00:00:00 2001
From: Richard Shiue <71320345+richardshiue@users.noreply.github.com>
Date: Mon, 23 Sep 2024 20:44:23 +0800
Subject: [PATCH] fix: semi-transparent colors being forced to full opacity in
 FlowySvg (#6376)

* fix: semi-transparent colors being forced to full opacity

* chore: code style

* chore: update comment
---
 .../packages/flowy_svg/lib/src/flowy_svg.dart     | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/frontend/appflowy_flutter/packages/flowy_svg/lib/src/flowy_svg.dart b/frontend/appflowy_flutter/packages/flowy_svg/lib/src/flowy_svg.dart
index a69a4bb90d..cba112dc2b 100644
--- a/frontend/appflowy_flutter/packages/flowy_svg/lib/src/flowy_svg.dart
+++ b/frontend/appflowy_flutter/packages/flowy_svg/lib/src/flowy_svg.dart
@@ -23,7 +23,7 @@ class FlowySvg extends StatelessWidget {
     this.size,
     this.color,
     this.blendMode = BlendMode.srcIn,
-    this.opacity = 1.0,
+    this.opacity,
     this.svgString,
   });
 
@@ -34,7 +34,7 @@ class FlowySvg extends StatelessWidget {
     Size? size,
     Color? color,
     BlendMode? blendMode = BlendMode.srcIn,
-    double opacity = 1.0,
+    double? opacity,
   }) {
     return FlowySvg(
       const FlowySvgData(''),
@@ -73,13 +73,16 @@ class FlowySvg extends StatelessWidget {
 
   /// The opacity of the svg
   ///
-  /// The default value is 1.0
-  final double opacity;
+  /// if null then use the opacity of the iconColor
+  final double? opacity;
 
   @override
   Widget build(BuildContext context) {
-    final iconColor =
-        (color ?? Theme.of(context).iconTheme.color)?.withOpacity(opacity);
+    Color? iconColor = color ?? Theme.of(context).iconTheme.color;
+    if (opacity != null) {
+      iconColor = iconColor?.withOpacity(opacity!);
+    }
+
     final textScaleFactor = MediaQuery.textScalerOf(context).scale(1);
 
     final Widget svg;