flutter / flutter/devtools

Audit scroll widgets in DevTools to see if notificationPredicate is needed

Open
#3,292 0 comments 0 reactions 0 assignees View on GitHub
cross-screen fix it friday P3
Dominant language
Dart
Stars
1.7k
Forks
404
Avg merge
6d 17h
Merged PRs (30d)
18

Description

A [`notificationPredicate`](https://api.flutter.dev/flutter/material/Scrollbar/notificationPredicate.html) is required for scrollbars whose corresponding scrollview is not the first nested scrollview beneath them (see image below for an example where the`notificationPredicate` is needed). DevTools wasn't using `notificationPredicate` before https://github.com/flutter/devtools/pull/3262. We should check to see if it's needed anywhere else.

![image](https://user-images.githubusercontent.com/21270878/129966770-a81042bc-e736-4c46-a735-0ce9b723d473.png)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.