flutter / flutter/devtools

Reducing Selection to only rely on nodeIndexCalculator and not on nodeIndex

Offen
#4,266 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
P3 screen: app size table component
Vorherrschende Sprache
Dart
Sterne
1.7k
Forks
404
Ø Merge
6 T. 17 Std.
Gemergte PRs (30 T.)
18

Beschreibung

**Currently**
Selection class in table.dart contains both nodeIndex and nodeIndexCalculator. https://github.com/flutter/devtools/blob/master/packages/devtools_app/lib/src/shared/table.dart#L321

**Future**
Only NodeIndexCalculator is needed and all changes to nodeIndex can be migrated to nodeIndexCalculator. Only changes of nodeIndex is found in memory_heap_tree.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in packages/devtools_app/lib/src/shared/table.dart, wo Selection derzeit nodeIndex und nodeIndexCalculator enthält. Untersuche anschließend memory_heap_tree auf jede Änderung von nodeIndex und verfolge, wie diese Aktualisierungen zu nodeIndexCalculator verschoben werden können. Erledigt ist es, wenn Selection nicht mehr auf nodeIndex angewiesen ist und der memory heap tree den Calculator konsistent verwendet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
dart
Bereich
devtools
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
52/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.