flutter / flutter/devtools

The devtools flame chart includes a large number of unneeded widgets

未关闭
#2,332 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
flame_chart P3 screen: cpu profiler
主要语言
Dart
星标
1.7k
派生
404
平均合并
6 天 17 小时
30 天内合并 PR
18

描述

There are a large number of empty Padding and SizedBox widgets in the flame chart. We should be able to optimize to eliminate all cases where we have 10 unneeded padding + sized boxes on a row without content. Technically we shouldn't need any padding or empty size boxes as the ExtentDelegateListView can define the spacing between widgets.
Screen Shot 2020-09-11 at 12 23 06 PM

Fyi @kenzieschmoll

贡献指南

打开贡献指南

调研方向

首先重现 issue 中显示的 DevTools flame chart,并定位生成 ExtentDelegateListView 周围空 Padding 和 SizedBox widget 的代码。跟踪当前添加间距的方式,并找出一行中有 10 个或更多空 widget 的情况。当这些不必要的 widget 被消除,同时 flame chart 布局仍然正确时,即视为完成。

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

评估

技术栈
dart, flutter
领域
devtools, performance
Issue 类型
重构
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

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