website
website copied to clipboard
After deprecation period, remove this recipe from website
Page URL
https://docs.flutter.dev/cookbook/effects/photo-filter-carousel/
Page source
https://github.com/flutter/website/tree/main/src/cookbook/effects/photo-filter-carousel.md
Describe the problem
This cookbook has several places to improve:
- codes and explanation are different from each other, as mentioned in #8202
-
Material
widget is unnessary. - In
_buildShadowGradient
,height: itemSize + ...
will be better thanheight: itemSize * 2 + ...
because theScrollable
will take less space on the screen. Ifheight: itemSize + ...
used, it will be better to modifyLinearGradient
’sColors.black
toColors.black87
to make the UI more comfortable. - The
Ring
has the with 6.0 while theFilterItem
has padding set to 8.0. It will be better if they are the same. - In
FilterSelctor
, current widget hierarchy isScrollable > LayoutBuilder > Stack
. It will be better to change it toLayoutBuilder > Stack > Scrollable
becauseScrollable
is not related with theRing
andShadow
. -
IgnorePointer
is useless becauseScrollable
has a higher position (upper in the widget hierarchy) thanSelctionRing
.
Expected fix
No response
Additional context
No response