flutter / flutter/devtools

Audit scroll widgets in DevTools to see if notificationPredicate is needed

Ouverte
#3,292 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
cross-screen fix it friday P3
Langage dominant
Dart
Étoiles
1.7k
Forks
404
Merge moyen
6 j 17 h
PR mergées (30 j)
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)

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.