Skip to content
This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

[Bug] There is a tiny gap between GrabbingWidget and SnappingSheetContent #76

Open
DarkMikey opened this issue Sep 7, 2021 · 2 comments

Comments

@DarkMikey
Copy link

DarkMikey commented Sep 7, 2021

If there is no separator between grabbing handle and bottom sheet you can see the background shine through. Would it be possible to fix this? :)

How to reproduce
I basically replaced the background of your example app with a red container and removed the separator in the handle.

image
image

flutter doctor -v
[√] Flutter (Channel stable, 2.2.3, on Microsoft Windows [Version 10.0.19043.1165], locale de-AT)
    • Flutter version 2.2.3 at C:\flutter
    • Framework revision f4abaa0735 (10 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\user\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Android Studio
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = C:\Program Files\Android\Android Studio
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.60.0)
    • VS Code at C:\Users\user\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.26.0

[√] Connected device (3 available)
    • Pixel 4a (mobile) • 0A191JEC216171 • android-arm64  • Android 11 (API 30)
    • Chrome (web)      • chrome         • web-javascript • Google Chrome 93.0.4577.63
    • Edge (web)        • edge           • web-javascript • Microsoft Edge 93.0.961.38

• No issues found!
@om-ha
Copy link

om-ha commented Oct 14, 2021

@AdamJonsson
Yes please that's an issue, any thoughts on how to circumvent this? Even a work-around maybe?

One work-around comes to mind applied from user-end, is getting not using grabber property altogether and incorporate the grabber widget into sheetBlow, thought I would love to keep using the grabber property.

@danoetz
Copy link

danoetz commented Mar 29, 2022

Just add solid white shadow at bottom of your grabbing widget

BoxShadow(
    blurRadius: 2,
    spreadRadius: 2,
    color: Colors.white,
    offset: Offset(0, 10),
)

tricky, but it's work 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants