Skip to content

Commit

Permalink
fix[example]: cupertino scrollbar (#13)
Browse files Browse the repository at this point in the history
  • Loading branch information
knopp authored Feb 18, 2024
1 parent 634f20e commit de9e17b
Showing 1 changed file with 21 additions and 5 deletions.
26 changes: 21 additions & 5 deletions example/lib/shell/app.dart
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import "package:flutter/material.dart"
ScrollbarThemeData,
Typography;
import "package:flutter/services.dart";
import "package:flutter/widgets.dart";
import "package:pixel_snap/widgets.dart";
import "package:provider/provider.dart";
import "package:super_sliver_list/super_sliver_list.dart";

Expand Down Expand Up @@ -225,6 +225,18 @@ class _ScrollBehavior extends ScrollBehavior {
parent: super.getScrollPhysics(context));
}

@override
TargetPlatform getPlatform(BuildContext context) {
// Cupertino scrollbar has broken overscroll, for now force macOS on
// all platforms.
final platform = super.getPlatform(context);
if (platform == TargetPlatform.iOS) {
return TargetPlatform.macOS;
} else {
return platform;
}
}

@override
Widget buildScrollbar(
BuildContext context, Widget child, ScrollableDetails details) {
Expand All @@ -238,10 +250,14 @@ class _ScrollBehavior extends ScrollBehavior {
return Colors.black.withOpacity(0.1);
}),
),
child: Scrollbar(
controller: details.controller,
thumbVisibility: true,
child: child,
child: MediaQuery(
data: MediaQuery.of(context)
.copyWith(platformBrightness: Brightness.light),
child: Scrollbar(
controller: details.controller,
thumbVisibility: true,
child: child,
),
),
);
}
Expand Down

0 comments on commit de9e17b

Please sign in to comment.