Browse Source

fix: color not show issue when change cover photo color. (#2624)

* fixe: color not show issue when change cover photo color.

* fix:added missing a required comma. and toColor() extension
chirag paneliya 1 year ago
parent
commit
57389d1fdd

+ 6 - 8
frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/cover/change_cover_popover.dart

@@ -499,27 +499,25 @@ class _CoverColorPickerState extends State<CoverColorPicker> {
       child: Padding(
         padding: const EdgeInsets.only(right: 10.0),
         child: SizedBox.square(
-          dimension: 25,
+          dimension: isChecked ? 24 : 25,
           child: Container(
             decoration: BoxDecoration(
-              color: isChecked
-                  ? Colors.transparent
-                  : Color(int.tryParse(option.colorHex) ?? 0xFFFFFFFF),
+              color: option.colorHex.toColor(),
               border: isChecked
                   ? Border.all(
-                      color: Color(int.tryParse(option.colorHex) ?? 0xFFFFFF),
+                      color: const Color(0xFFFFFFFF),
+                      width: 2.0,
                     )
                   : null,
               shape: BoxShape.circle,
             ),
             child: isChecked
                 ? SizedBox.square(
-                    dimension: 25,
+                    dimension: 24,
                     child: Container(
                       margin: const EdgeInsets.all(4),
                       decoration: BoxDecoration(
-                        color:
-                            Color(int.tryParse(option.colorHex) ?? 0xFFFFFFFF),
+                        color: option.colorHex.toColor(),
                         shape: BoxShape.circle,
                       ),
                     ),