Skip to content

Shadows (and other paintBounds-exceeding paints) are dropped by lazy lists #170

Closed

Description

@FlutterIssues

Issue by Hixie
Thursday Sep 24, 2015 at 19:07 GMT
Originally opened as https://github.com/flutter/engine/issues/1323


Suppose you have a widget whose position & size happens to place it so that its bottom is just off the top of a vertically scrolling list. Its shadow, if any, will vanish.

We should base these widgets on the children's paintBounds, not their sizes.

Activity

  1. added this to the milestone on Nov 9, 2015
  2. added
    frameworkflutter/packages/flutter repository. See also f: labels.
    on Nov 9, 2015
  3. abarth commented on Mar 30, 2016

    @abarth
    Contributor

    Paint bounds isn't a thing anymore. Maybe the solution here is to pad the viewport by one item? It's hard to know how far a shadow will extend from beyond the viewport...

  4. Hixie commented on Aug 17, 2016

    @Hixie
    Contributor

    #5467 would provide a real fix for this, but we should probably do the padding thing.

  5. self-assigned this
    on Sep 23, 2016
  6. removed their assignment
    on Mar 9, 2017
  7. changed the title [-]Shadows (and other paintBounds-exceeding paints) are dropped by HomogeneousViewport and MixedViewport for off-screen widgets[/-] [+]Shadows (and other paintBounds-exceeding paints) are dropped by lazy lists[/+] on May 30, 2017
  8. self-assigned this
    on Jun 23, 2017
  9. Hixie commented on Jun 26, 2017

    @Hixie
    Contributor

    A bunch of bugs about scrolling laziness claim that this is the fix, but actually a better fix for this is pad the viewport by one on each side. We'll also need that to fix #1714.

  10. assigned and unassigned on Jun 26, 2017
  11. added a commit that references this issue on Mar 21, 2019
  12. github-actions commented on Aug 17, 2021

    @github-actions

    This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

  13. locked as resolved and limited conversation to collaborators on Aug 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

f: scrollingViewports, list views, slivers, etc.frameworkflutter/packages/flutter repository. See also f: labels.p: material_uimaterial_ui package in flutter/packages

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions