feat: Add ink parameter to CheckboxItem and update related screens

Robert-Stackflow · Robert-Stackflow · commit 89844d90a874 · 2026-05-23T23:57:18.000+08:00
diff --git a/lib/Screens/Backup/aliyundrive_service_screen.dart b/lib/Screens/Backup/aliyundrive_service_screen.dart
@@ -194,6 +194,7 @@ class _AliyunDriveServiceScreenState
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeAliyunDrive,
         description: appLocalizations.cloudOAuthSafeTip(
             CloudService.serverEndpoint, appLocalizations.cloudTypeAliyunDrive),
diff --git a/lib/Screens/Backup/box_service_screen.dart b/lib/Screens/Backup/box_service_screen.dart
@@ -202,6 +202,7 @@ class _BoxServiceScreenState extends BaseDynamicState<BoxServiceScreen>
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeBox,
         description: appLocalizations.cloudOAuthSafeTip(
             CloudService.serverEndpoint, appLocalizations.cloudTypeBox),
diff --git a/lib/Screens/Backup/dropbox_service_screen.dart b/lib/Screens/Backup/dropbox_service_screen.dart
@@ -191,6 +191,7 @@ class _DropboxServiceScreenState extends BaseDynamicState<DropboxServiceScreen>
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeDropbox,
         value: _dropboxCloudServiceConfig?.enabled ?? false,
         onTap: () {
diff --git a/lib/Screens/Backup/googledrive_service_screen.dart b/lib/Screens/Backup/googledrive_service_screen.dart
@@ -187,6 +187,7 @@ class _GoogleDriveServiceScreenState
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeGoogleDrive,
         description: appLocalizations.cloudOAuthSafeTip(
             CloudService.serverEndpoint, appLocalizations.cloudTypeGoogleDrive),
diff --git a/lib/Screens/Backup/huawei_service_screen.dart b/lib/Screens/Backup/huawei_service_screen.dart
@@ -203,6 +203,7 @@ class _HuaweiCloudServiceScreenState
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeHuaweiCloud,
         description: appLocalizations.cloudOAuthSafeTip(
             CloudService.serverEndpoint, appLocalizations.cloudTypeHuaweiCloud),
diff --git a/lib/Screens/Backup/onedrive_service_screen.dart b/lib/Screens/Backup/onedrive_service_screen.dart
@@ -199,6 +199,7 @@ class _OneDriveServiceScreenState
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeOneDrive,
         description: appLocalizations.cloudTypeOneDriveTip,
         value: _oneDriveCloudServiceConfig?.enabled ?? false,
diff --git a/lib/Screens/Backup/s3_service_screen.dart b/lib/Screens/Backup/s3_service_screen.dart
@@ -178,6 +178,7 @@ class _S3CloudServiceScreenState extends BaseDynamicState<S3CloudServiceScreen>
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeS3Cloud,
         value: _s3CloudServiceConfig?.enabled ?? false,
         onTap: () {
diff --git a/lib/Screens/Backup/webdav_service_screen.dart b/lib/Screens/Backup/webdav_service_screen.dart
@@ -170,6 +170,7 @@ class _WebDavServiceScreenState extends BaseDynamicState<WebDavServiceScreen>
     return Container(
       padding: const EdgeInsets.symmetric(horizontal: 6),
       child: CheckboxItem(
+        ink: false,
         title: appLocalizations.enable + appLocalizations.cloudTypeWebDav,
         value: _webDavCloudServiceConfig?.enabled ?? false,
         onTap: () {
diff --git a/lib/Screens/layout_select_screen.dart b/lib/Screens/layout_select_screen.dart
@@ -182,7 +182,7 @@ class _LayoutSelectScreenState extends BaseDynamicState<LayoutSelectScreen> {
           borderRadius: BorderRadius.circular(12),
           border: Border.all(
             color: selected ? _accent : ChewieTheme.borderColor,
-            width: selected ? 1.5 : 1,
+            width: selected ? 1.5 : 0,
           ),
         ),
         padding: const EdgeInsets.all(10),
diff --git a/lib/Screens/sort_select_screen.dart b/lib/Screens/sort_select_screen.dart
@@ -232,7 +232,6 @@ class _SortSelectScreenState extends BaseDynamicState<SortSelectScreen> {
           decoration: BoxDecoration(
             color: ChewieTheme.canvasColor,
             borderRadius: BorderRadius.circular(12),
-            border: Border.all(color: ChewieTheme.borderColor),
           ),
           child: Column(
             mainAxisSize: MainAxisSize.min,
diff --git a/lib/Screens/welcome_screen.dart b/lib/Screens/welcome_screen.dart
@@ -444,7 +444,7 @@ class _WelcomeScreenState extends State<WelcomeScreen>
   Widget _buildSkipButton() {
     return Positioned(
       top: 8,
-      right: ResponsiveUtil.isMacOS() ? 8 : 16,
+      right: 8,
       child: TextButton(
         onPressed: _skip,
         style: TextButton.styleFrom(
diff --git a/third-party/chewie/lib/src/Widgets/Tile/checkbox_item.dart b/third-party/chewie/lib/src/Widgets/Tile/checkbox_item.dart
@@ -16,9 +16,11 @@ class CheckboxItem extends SearchableStatefulWidget {
   final double trailingLeftMargin;
   final double padding;
   final bool disabled;
+  final bool ink;
 
   const CheckboxItem({
     super.key,
+    this.ink = true,
     this.radius = 8,
     this.roundTop = false,
     this.roundBottom = false,
@@ -57,6 +59,7 @@ class CheckboxItem extends SearchableStatefulWidget {
       trailingLeftMargin: trailingLeftMargin,
       padding: padding,
       disabled: disabled,
+      ink: ink,
     );
   }
 
@@ -68,10 +71,11 @@ class CheckboxItemState extends SearchableState<CheckboxItem> {
   double get _effectivePadding =>
       widget.description.isNotEmpty ? widget.padding : widget.padding - 3;
 
-  BorderRadius get _borderRadius => BorderRadius.vertical(
+  BorderRadius get _borderRadius =>
+      BorderRadius.vertical(
         top: widget.roundTop ? Radius.circular(widget.radius) : Radius.zero,
         bottom:
-            widget.roundBottom ? Radius.circular(widget.radius) : Radius.zero,
+        widget.roundBottom ? Radius.circular(widget.radius) : Radius.zero,
       );
 
   @override
@@ -80,7 +84,7 @@ class CheckboxItemState extends SearchableState<CheckboxItem> {
     if (!shouldShow) return const SizedBox.shrink();
     return InkAnimation(
       borderRadius: _borderRadius,
-      ink: true,
+      ink: widget.ink,
       color: Colors.transparent,
       onTap: widget.disabled ? null : () {
         HapticFeedback.lightImpact();
@@ -175,9 +179,9 @@ class CheckboxItemState extends SearchableState<CheckboxItem> {
           onChanged: widget.disabled
               ? null
               : (_) {
-                  HapticFeedback.lightImpact();
-                  widget.onTap?.call();
-                },
+            HapticFeedback.lightImpact();
+            widget.onTap?.call();
+          },
         ),
       ),
     );
