-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Description
This may be a docs/sample bug. I'm aware of #154618, but my main concern is that the web demo for https://api.flutter.dev/flutter/material/CarouselView-class.html does not work on the web.
When I saw the sample didn't work, I searched the page for any mention of "web" and didn't find any, so I was about to log a bug but found #154618 in a search.
I think the sample should include the workaround provided in the comments for #154618, i.e. the scrollBehavior: MyCustomScrollBehavior() code, so that the sample works and devs don't have to hunt for a comment in a bug to make CarouselView work on the web (and probably other systems that are mouse-based). Ideally the text could mention MaterialScrollBehavior and explain that this is necessary for mouse-based systems.
Even better, it would be nice if CarouselView just worked on all platforms without that scrollBehavior: MyCustomScrollBehavior() patch to MaterialApp, but I recognize there may be good reasons why it doesn't.