flutter / flutter/devtools

Audit scroll widgets in DevTools to see if notificationPredicate is needed

未关闭
#3,292 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
cross-screen fix it friday P3
主要语言
Dart
星标
1.7k
派生
404
平均合并
6 天 17 小时
30 天内合并 PR
18

描述

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)

贡献指南

打开贡献指南

调研方向

在 DevTools 代码库中搜索 Scrollbar 和嵌套 scrollview 的使用情况,然后将其与 issue 中链接的 notificationPredicate API 指南进行比较。完成的标准是检查过每个相关的 Scrollbar,并识别和处理所有需要 notificationPredicate 的情况。

由索引模型根据 Issue 内容生成。

评估

技术栈
dart
领域
devtools
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。